Skip to content
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

add hint to ActionView's fields_for #44093

Merged
merged 1 commit into from
Feb 10, 2022

Conversation

motine
Copy link
Contributor

@motine motine commented Jan 6, 2022

Summary

Add a hint to the documentation for ActionView::Helpers::FormHelper#fields_for when hidden id fields are generated.

Other Information

I am a little embarrassed that this is such a small change. This missing bit of documentation had me spend unnecessary time when trying to figure out why there was no id rendered for sub-forms.

@rails-bot rails-bot bot added the actionview label Jan 6, 2022
@motine
Copy link
Contributor Author

motine commented Jan 10, 2022

there seem to be quite some errors in the CI. please let me know if my documentation change caused any of this.

@ghiculescu
Copy link
Member

there seem to be quite some errors in the CI. please let me know if my documentation change caused any of this.

No, it seems unrelated. Could you rebase against main to re-run CI?

@motine
Copy link
Contributor Author

motine commented Feb 8, 2022

sure thing. please I just merged upstream.

@ghiculescu ghiculescu added the docs label Feb 8, 2022
@ghiculescu
Copy link
Member

Thanks, could you also squash your commits please?

@motine
Copy link
Contributor Author

motine commented Feb 8, 2022

alrighty, please check again.

@ghiculescu ghiculescu added the ready PRs ready to merge label Feb 8, 2022
@rafaelfranca rafaelfranca merged commit b961af3 into rails:main Feb 10, 2022
rafaelfranca added a commit that referenced this pull request Feb 10, 2022
@nievesmiguel
Copy link

gh pr checkout 13271

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
actionview docs ready PRs ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants