Skip to content

Supports ActiveRecord >= 7#2527

Open
lucasgomide wants to merge 2 commits intojonhue:mainfrom
lucasgomide:main
Open

Supports ActiveRecord >= 7#2527
lucasgomide wants to merge 2 commits intojonhue:mainfrom
lucasgomide:main

Conversation

@lucasgomide
Copy link
Copy Markdown

@lucasgomide lucasgomide commented Mar 3, 2025

Address this issue

ActiveRecord 7 introduced a change in the serialize method, requiring
the coder: keyword argument instead of the previous syntax. This commit
adds version detection to apply the correct serialize syntax dynamically.

This ensures the library remains compatible across different versions of
ActiveRecord without breaking older setups.

@Leonardo-Zappani
Copy link
Copy Markdown

UP

1 similar comment
@ricardogontarz-subbase
Copy link
Copy Markdown

UP

@nardocesar
Copy link
Copy Markdown

Up!

@thiagobonfante
Copy link
Copy Markdown

+1

2 similar comments
@charlieslin
Copy link
Copy Markdown

+1

@jeanlondero
Copy link
Copy Markdown

+1

@lucasgomide
Copy link
Copy Markdown
Author

@jonhue is ready to review. Can you review that?

Some .sqlite files were added after running rspec
ActiveRecord 7 introduced a change in the `serialize` method, requiring
the `coder:` keyword argument instead of the previous syntax. This commit
adds version detection to apply the correct `serialize` syntax dynamically.

This ensures the library remains compatible across different versions of
ActiveRecord without breaking older setups.
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.

7 participants