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

ISO3166 namespace similar to ISO4217 used in currencies gem + fixes related to currencies 0.4.0 #1

Merged
6 commits merged into from
Feb 14, 2011

Conversation

doubledrones-grid002
Copy link

We introduced ISO3166 namespace for countries gem similar to ISO4217 used in currencies gem.

There is also fix to support currencies 0.4.0 gem in countries properly.

countries gem has version 0.4.0 to match currencies with similar functionalities.

Marcin Michalowski and Marcin Nowicki added 6 commits February 8, 2011 08:25
As in "currencies" gem prior to 0.3.0 we use namespace ISO4217 to
prevent name collisions.
This also allow test to pass when using "currencies" in 0.4.0 version.
From now default behavior is to use ISO3166::Country.

Country class is still there for backward compatibility.
It is inherited from ISO3166::Country.

Version bumped to 0.4.0 to match currencies gem with similar
functionality.
Also speeds up and give "lazy" usage of "country".
It allow to find by name and name aliases.
@doubledrones-grid002
Copy link
Author

We add some changes:

Clean some code.

Add .find_by_name method useful for searching via name aliases.

@hexorx
Copy link
Collaborator

hexorx commented Feb 14, 2011

Code committed and a new gem pushed out. Once again, thank you for your contributions to the project.

hexorx pushed a commit that referenced this pull request Mar 27, 2013
rposborne pushed a commit that referenced this pull request Nov 6, 2015
pmor pushed a commit that referenced this pull request Dec 15, 2021
Updates for data yaml files
This pull request was closed.
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