Skip to content
This repository has been archived by the owner on Jun 10, 2022. It is now read-only.

merge upstream #2

Open
wants to merge 154 commits into
base: master
Choose a base branch
from
Open

merge upstream #2

wants to merge 154 commits into from

Conversation

dalyons
Copy link

@dalyons dalyons commented Jul 31, 2019

our forked version is way out of date.

mordaroso and others added 30 commits March 20, 2018 10:22
…ized-attribs

Exclude path params from attributes hash
It is very happy if we can write `Model.page(params[:page])` like Kaminari.
…-nil

allow nil for page number parameter
Fix custom type comment to use TypeFactory
…-to-fix-empty-array-param-issue

JsonApiClient#278 updating faraday to fix issue when param value is an empty array
…eate-and-update-resource

allow to send fields and/or includes on create/update resource
…arning

Use mocha/minitest instead of mocha/mini_test on test_helper
…r-new-custom-connection-arguments

Update arguments for custom connections run method in README
fixing readme to update test and coverage icon targets
it will fix behavior when relationships and links are passed as symbol keys
…-pagination-override-example

JsonApiClient#291 import README - pagination override examples
…w-symbolized-relationship-key

JsonApiClient#290 symbolize params keys on model initialize
@sirwolfgang sirwolfgang mentioned this pull request Aug 5, 2019
yahonda and others added 25 commits August 17, 2019 03:22
…nt-formatter

fix missing constant formatter
…esource

Add an immutable option to JSON API resources.
…edMethod

call super from inherited method so that it will execute parent classes' implementations
Relax faraday and faraday_middleware version
BigDecimal 2.0 no longer supports `.new`.  looks like this gem uses both `BigDecimal(x)` and `BigDecimal.new(x)` - mind if we upgrade these last few calls?
Ruby 2.7 deprecates automatic conversion from a hash to keyword arguments
https://blog.saeloun.com/2019/10/07/ruby-2-7-keyword-arguments-redesign.html

fixing those up, plus one more I noticed in the logs from JsonApiClient#370 (comment)

https://travis-ci.org/github/JsonApiClient/json_api_client/jobs/722402404

thanks @sharshenov
@tilo
Copy link

tilo commented Sep 26, 2020

@dpep we should merge this ... eventually (probably after AWS is live)

Tilo Sloboda and others added 2 commits September 25, 2020 22:39
Merge remote-tracking branch 'upstream/master' into merge-more-upstream
Merge more upstream changes into Dave's branch
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.