Fix for Rails 8.1 - #1494
Closed
StephenHulme wants to merge 43 commits into
Closed
Conversation
* If caching has been enabled, however disabled for a resource, the
context values are nested in a a 'context' key. For example:
{ context: { foo: :bar } } vs { foo: :bar }
(cherry picked from commit 0936916)
(cherry picked from commit 2f46f58)
(cherry picked from commit fd0826a)
Update README to use Doc Site
…ation with partially preloaded relations, resolves JSONAPI-Resources#889 (cherry picked from commit d9d9d3f)
…n `_parser_exception` (cherry picked from commit e144f80)
(cherry picked from commit 854873c)
Fix 933 - Sort with has one include
Fixes an issue where the recreated relationships might be using the wrong model_name if the model_name is changed in a resource that is derived from a non abstract resource, such as done in many of the tests. Breaking change: Derived resources now use the model name of their base resource, if it is not abstract. Cleans up the tests to not have the warnings about missing models. Fixes issue in _model_class related to JSONAPI-Resources#952, and renames @model (class level) to @model_class to avoid confusion with instance level @model Fixes JSONAPI-Resources#952, JSONAPI-Resources#654 (cherry picked from commit 280ccea)
* Added test and potential fix for JSONAPI-Resources#628. * Add handling for CSV parsing errors. (cherry picked from commit 9306ec0)
* Fix has_one polymorphism * Add tests for has one polymorphic serialization (cherry picked from commit 9ad4d91)
(cherry picked from commit 31d22d1)
…ation with partially preloaded relations, resolves JSONAPI-Resources#889 (cherry picked from commit d9d9d3f)
Fixes JSONAPI-Resources#960 Potentially a breaking change for apps with guids (cherry picked from commit 4f51331)
(cherry picked from commit 57fa0ae)
(cherry picked from commit 48eaa02)
(cherry picked from commit 4b93707)
… update description for clarity
…i-resources gem to ensure compatibility with RubyGems and Bundler
…7.1.0 which are required for compatibility with Rails 7.1 and Ruby 3.x
…and < 3.0 (ensuring compatibility with both older and newer Ruby/Rails setups, while avoiding breaking changes introduced in Bundler 3.x.) ; add runtime dependency for csv (because Ruby 3.4+ no longer includes the csv library by default)
…n controller and resource generators
…recent Rails versions, replacing deprecated or invalid patterns to ensure proper table and index creation.
… with Ruby and Rails testing
…ownership and repository link
Y25-318 Publish gem as sanger-jsonapi-resources
…-to-fix-the-json-api-resource-incompatibility-with-rack-which-is-causing-422-errors-to-fail-with-a-status-of-0 Y25-319 - [PR] [Develop] Add compatibility for obsolete Rack status symbols
Adding CONTRIBUTING.md Tests are failing but nothing to do with the changes to .md files here.
…test API changes Based on merge pull request #10 from sanger/rails-8.1-fix
Author
|
Wrong fork - sorry for the noise! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
16 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fix for Rails 8.1