Skip to content
This repository has been archived by the owner on Aug 17, 2017. It is now read-only.

Failing test for fields_for style nested attributes. #24

Closed
wants to merge 2 commits into from
Closed

Failing test for fields_for style nested attributes. #24

wants to merge 2 commits into from

Conversation

erichmenge
Copy link
Contributor

Here is a failing test for a fields_for style nested attributes hash. Maybe this is expected behavior and there is an alternative.

I don't often use accepts_nested_attributes_for but I can understand the confusion regarding this behavior.

What do you think @dhh?

@erichmenge
Copy link
Contributor Author

Relates to #16

@erichmenge
Copy link
Contributor Author

Adding one possible solution as well.

@erichmenge
Copy link
Contributor Author

@dhh I put your name in the pull request, but I'm not sure if it pings you if you include someone's name in the actual PR description. I wanted to make sure you saw this. My apologies if you just haven't gotten to it.

@dhh
Copy link
Member

dhh commented Jun 29, 2012

Seems like strong_parameters should play nice. I never use nested params, so hopefully someone else from core can have a look when you come up with an implementation. I just want to make sure there are no side effects.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants