Replies: 1 comment
-
The documentation for |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Hi Jeremy,
I ran into an issue today where I was testing validations and was surprised that calling
save
raised an error on invalid records rather than returningfalse
as in AR. I was confused because I did not think I had setraise_on_save_failure
to true, and could not find that anywhere in my codebase. Then I dug around the docs a while and finally realized that this defaults to true. So, nothing wrong, I just remembered the default incorrectly.I did want to offer a small suggestion for you to consider...
The first docs I checked were the save method.
Those docs are clear that
raise_on_save_failure
determines the default, but they don't mention what the default value for that setting is, nor do they link to the docs for it.Would it make sense to you to either add mention of the default value on the
save
docs, or add a link toraise_on_save_failure
?Beta Was this translation helpful? Give feedback.
All reactions