-
Notifications
You must be signed in to change notification settings - Fork 26
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
Rails 7.1 fix: use SelectRenderer for add_options and include FormOptionsHelper for … #148
Rails 7.1 fix: use SelectRenderer for add_options and include FormOptionsHelper for … #148
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution! ❤️ I've added my review comments.
…currency_options_for_select
4c7b911
to
e072788
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Getting close … now let’s make RuboCop happy. :-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Thanks a lot, @hishammalik! ❤️
Sorry, this got merged to the wrong branch, which I accidentally overwrote. I'm going to manually re-apply your changes in a minute. Sorry for this. @hishammalik For the future, I'd like to ask you to only do PRs against a project's main branch (which usually is |
I've re-applied your changes now in #151. |
Hi @hishammalik, thank you so much again for contributing! ❤️ I've just released version 6.0.0 with your changes: https://github.com/braingourmets/currency_select/releases/tag/v6.0.0 |
…currency_options_for_select