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

Use #to_hash instead of #as_json #44

Merged

pyromaniac merged 1 commit into main from use-to_hash

Jan 1, 2024

Contributor

pyromaniac commented Jan 1, 2024 •

edited

Loading

This hook is more correct to use https://github.com/rails/rails/blob/a8dcfbde95b681c9e6f324bccd80f374f30baeff/activesupport/lib/active_support/core_ext/object/json.rb#L59-L65


          Use #to_hash instead of #as_json

2d0978d

pyromaniac force-pushed the use-to_hash branch from 44571f1 to 2d0978d Compare

January 1, 2024 10:03

pyromaniac merged commit 2cb9d5a into main

6 checks passed

pyromaniac deleted the use-to_hash branch

January 1, 2024 10:49

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet