Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

can't modify frozen Array #365

Open
dapi opened this issue Dec 21, 2018 · 0 comments
Open

can't modify frozen Array #365

dapi opened this issue Dec 21, 2018 · 0 comments

Comments

@dapi
Copy link

dapi commented Dec 21, 2018

Error raised when we pass frozen array to include option.

fast_jsonapi/lib/fast_jsonapi/object_serializer.rb

Line 89 in fdcaed6

@includes = options[:include].delete_if(&:blank?).map(&:to_sym)

y0nigt pushed a commit to seekingalpha/fast_jsonapi that referenced this issue Jan 10, 2019
@dapi @y0nigt
[Netflix#365] Support frozen array in option
07d4aa9
shishirmk pushed a commit that referenced this issue Mar 29, 2019
@dapi @shishirmk
[#365] Support frozen array in option
36b8ea2
ThomasSevestre pushed a commit to ThomasSevestre/fast_jsonapi that referenced this issue Sep 13, 2019
@dapi
[Netflix#365] Support frozen array in option
a7fba78
ThomasSevestre pushed a commit to ThomasSevestre/fast_jsonapi that referenced this issue Sep 13, 2019
@dapi
[Netflix#365] Support frozen array in option
8d8eaec
kpheasey referenced this issue in jsonapi-serializer/jsonapi-serializer Oct 4, 2019
@dapi @kpheasey
[#365] Support frozen array in option
1a407c0
listrophy pushed a commit to listrophy/fast_jsonapi that referenced this issue Oct 8, 2019
@dapi
[Netflix#365] Support frozen array in option
b0bb2e7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@dapi