Skip to content

Document the child_index option of simple_fields_for#418

Closed
Unpakt wants to merge 1 commit into
heartcombo:masterfrom
Unpakt:master
Closed

Document the child_index option of simple_fields_for#418
Unpakt wants to merge 1 commit into
heartcombo:masterfrom
Unpakt:master

Conversation

@Unpakt

@Unpakt Unpakt commented Jan 26, 2012

Copy link
Copy Markdown

Hi. We had a need for this option and we did not know it existed. After some digging in rails and simple_form, we found reference to child_index in simple_form's test suite. We felt it would be nice to have on the Readme so that other people could find the solution without digging through the source.

@josevalim

Copy link
Copy Markdown
Contributor

Thanks, but just as note, this is a Rails option available in fields_for as well. :)

@rafaelfranca

Copy link
Copy Markdown
Collaborator

I think that is better to explain that simple_fields_for accepts the same options that fields_for instead of this changes.

@josevalim

Copy link
Copy Markdown
Contributor

@Unpakt if you haven't found those notes in Rails docs, maybe you should update them as well. You can push directly to github.com/lifo/docrails

@josevalim

Copy link
Copy Markdown
Contributor

Closing this since it is a rails option. :)

@josevalim josevalim closed this Jan 27, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants