Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Handle form object being false #173

Merged

Conversation

mcasper
Copy link
Contributor

@mcasper mcasper commented Oct 3, 2024

As of rails 7.2 a form's object defaults to false rather than nil, breaking any of the methods in this library that eventually call object_errors. This change expands object_errors to handle the new default value for object, returning nil as one would expect.

Rails PR reference: rails/rails#49943

@mcasper mcasper force-pushed the method-errors-form-object-false branch from 9764a6a to 517cfce Compare October 3, 2024 17:52
@Spone Spone requested a review from nicolas-brousse October 8, 2024 09:26
Copy link
Collaborator

@Spone Spone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thanks @mcasper!

@nicolas-brousse
Copy link
Member

Thank you @mcasper

@nicolas-brousse nicolas-brousse merged commit 149ae39 into pantographe:main Oct 8, 2024
20 of 72 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants