Skip to content

Revert "Drop support for MultiJson (#124)" - #125

Open
OJFord wants to merge 1 commit into
masterfrom
undrop-multijson
Open

Revert "Drop support for MultiJson (#124)"#125
OJFord wants to merge 1 commit into
masterfrom
undrop-multijson

Conversation

@OJFord

@OJFord OJFord commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

It turns out we are presently still reliant (implicitly) on MultiJson internally; and Oj (the adapter used) has different behaviour to stdlib/json_gem in that TimeWithTimezone is serialised in ISO8601 format rather than the subtly different (and non-compliant) format used by its #to_s, which is used by the latter for objects other than special cases like arrays.

This reverts commit a35d85a, reversing changes made to 1c83073.

It turns out we _are_ presently still reliant (implicitly) on MultiJson
internally; and Oj (the adapter used) has different behaviour to
stdlib/`json_gem` in that `TimeWithTimezone` is serialised in ISO8601
format rather than the subtly different (and non-compliant) format used
by its `#to_s`, which is used by the latter for objects other than
special cases like arrays.

This reverts commit a35d85a, reversing
changes made to 1c83073.
Comment thread CHANGELOG.md
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.

2 participants