Skip to content

fix(ObjectSerializer): pass to_json arguments - #80

Merged
kpheasey merged 2 commits into
masterfrom
fix/to-json-arguments
May 1, 2020
Merged

fix(ObjectSerializer): pass to_json arguments#80
kpheasey merged 2 commits into
masterfrom
fix/to-json-arguments

Conversation

@kpheasey

@kpheasey kpheasey commented May 1, 2020

Copy link
Copy Markdown
Contributor

What is the current behavior?

ObjectSerializer#serialize_json does not accept arguments for to_json alias

What is the new behavior?

ObjectSerializer#serialize_json accepts arguments for to_json alias.

Note the allowed arguments in the original to_json https://apidock.com/rails/Hash/to_json

Checklist

Please make sure the following requirements are complete:

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes /
    features)
  • All automated checks pass (CI/CD)

to_json takes arguments that serialized_json did not define.  This breaks a lot of things.
@kpheasey
kpheasey requested a review from stas May 1, 2020 12:59
@kpheasey
kpheasey merged commit b64a0ce into master May 1, 2020
@kpheasey
kpheasey deleted the fix/to-json-arguments branch May 1, 2020 13:02
kpheasey added a commit that referenced this pull request May 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant