-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fields.nested_form[:update_only] should always be false by default. #2626
Comments
@antoine-lizee as a first step, I've created a gist to reproduce the issue here: https://gist.github.com/ericcf/f4da02ca5bb80d889131f603448ee10c There is a difference between @sferik is this interpretation correct? Would you like a failing spec for this case or is more discussion necessary? Thanks. |
We're having the same issues because of the misunderstanding of this option by Rails Admin. It's unexpected that you can't create a new association for a model when |
BTW, if you do the following:
In order to get Rails Admin to show the child form when creating a instance, you will actually change the settings on the model as You should do this instead:
|
See the "first bug" in this stack overflow answer.
It's a personal opinion, but I think that interpreting
accept_nested_attributes_for: ..., update_only:true
as "disable the form when creating the object" is a potential misunderstanding of the option, (for Rails 4), and should at least be documented in the wiki?The text was updated successfully, but these errors were encountered: