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

Replace all usage of alpha3/admin0/admin1 with country_a/country/region #6

Merged
merged 3 commits into from
Feb 29, 2016

Conversation

dianashk
Copy link
Contributor

  • rename things with alpha3, admin0, and admin1 to country_a, region, and region_a respectively
  • change all abbreviation properties to end with _a instead of _abbr in order to better match the Pelias-API result formatting

Connected to pelias/api#424

@dianashk dianashk self-assigned this Feb 29, 2016
@dianashk dianashk added this to the Who's on First milestone Feb 29, 2016
// { 'type': { 'value': 'local_admin' } },
// { 'type': { 'value': 'neighborhood' } }
// { 'type': { 'value': 'localadmin' } },
// { 'type': { 'value': 'neighbourhood' } }
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gotta keep that commented out code up to date :P

@orangejulius
Copy link
Member

👍

@dianashk
Copy link
Contributor Author

$ npm publish
+ [email protected]

dianashk pushed a commit that referenced this pull request Feb 29, 2016
Replace all usage of alpha3/admin0/admin1 with country_a/country/region
@dianashk dianashk merged commit 447b35b into master Feb 29, 2016
@orangejulius orangejulius deleted the abbr-to-a branch May 16, 2016 20:22
orangejulius added a commit to pelias/schema that referenced this pull request Nov 3, 2018
We stopped using the `alpha3` field over [two years ago](pelias/query#6),
but it was still lingering in our schema including some tests.

This was discovered while updating and testing #282.
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