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

Rails 7.1 fix: use SelectRenderer for add_options and include FormOptionsHelper for … #148

Merged
merged 2 commits into from
Dec 13, 2023

Conversation

hishammalik
Copy link

…currency_options_for_select

Copy link
Member

@oliverklee oliverklee left a 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.

lib/to_currency_select_tag.rb Outdated Show resolved Hide resolved
Copy link
Member

@oliverklee oliverklee left a 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. :-)

lib/to_currency_select_tag.rb Show resolved Hide resolved
Copy link
Member

@oliverklee oliverklee left a 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! ❤️

@oliverklee oliverklee merged commit 9c286a7 into braingourmets:feature/rails-71 Dec 13, 2023
10 checks passed
@oliverklee
Copy link
Member

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 main oder master) to make sure your changes don't get lost.

@oliverklee
Copy link
Member

I've re-applied your changes now in #151.

@oliverklee oliverklee mentioned this pull request Dec 13, 2023
@oliverklee
Copy link
Member

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

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

Successfully merging this pull request may close these issues.

2 participants