Skip to content

Nit: add explicit Oj Rails config for JSON serialisation#6876

Merged
compwron merged 2 commits intorubyforgood:mainfrom
alanparmenter:nit/oj-config-clean
Apr 24, 2026
Merged

Nit: add explicit Oj Rails config for JSON serialisation#6876
compwron merged 2 commits intorubyforgood:mainfrom
alanparmenter:nit/oj-config-clean

Conversation

@alanparmenter
Copy link
Copy Markdown
Contributor

What github issue is this PR for, if any?

No associated issue; small configuration addition to prevent implicit Oj Rails initialization warning, visible when testing.

What changed, and why?

Explicit configuration added to config/initializers in response to warning message when running sessions_spec test.

How is this tested? (please write rspec and jest tests!) 💖💪

bundle exec rspec spec/requests/api/v1/users/sessions_spec.rb

Screenshots please :)

Before:
Screenshot 2026-04-22 at 02 41 28
After:
Screenshot 2026-04-22 at 02 50 47

Feelings gif (optional)

What gif best describes your feeling working on this issue? https://giphy.com/
How to embed:

Great British Bakeoff five minute warning

Explicitly initialise Oj Rails integration
@github-actions github-actions Bot added the ruby Pull requests that update Ruby code label Apr 22, 2026
@compwron compwron merged commit ebb7718 into rubyforgood:main Apr 24, 2026
11 checks passed
@alanparmenter alanparmenter deleted the nit/oj-config-clean branch April 24, 2026 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ruby Pull requests that update Ruby code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants