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

Fix country_options_for_select for when selected = nil (the default) #44

Merged
merged 1 commit into from
Mar 1, 2013

Conversation

rymai
Copy link
Contributor

@rymai rymai commented Feb 25, 2013

Hi!

When selected (btw renamed selected_country_code),
ISO3166::Country[selected_country_code] would return false, hence
false.name was failing.

Thank you!

When selected (btw renamed selected_country_code),
ISO3166::Country[selected_country_code] would return false, hence
false.name was failing.
hexorx added a commit that referenced this pull request Mar 1, 2013
Fix country_options_for_select for when selected = nil (the default)
@hexorx hexorx merged commit 71c62d7 into countries:master Mar 1, 2013
@rymai
Copy link
Contributor Author

rymai commented Mar 1, 2013

Thank you!

@rymai rymai deleted the fix_country_select branch March 1, 2013 07:36
@rymai
Copy link
Contributor Author

rymai commented Mar 1, 2013

Any plan to release the 0.9.2 soon? :)

@hexorx
Copy link
Collaborator

hexorx commented Mar 1, 2013

I have been traveling but should have time to push an update tonight. Thank you for your contributions!

Sent from Mailbox for iPhone

On Fri, Mar 1, 2013 at 3:24 AM, Rémy Coutable notifications@github.com
wrote:

Any plan to release the 0.9.2 soon? :)

Reply to this email directly or view it on GitHub:
#44 (comment)

@hexorx
Copy link
Collaborator

hexorx commented Mar 2, 2013

done!

@rymai
Copy link
Contributor Author

rymai commented Mar 3, 2013

Awesome thank you!

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

Successfully merging this pull request may close these issues.

2 participants