Added qs:pop and mz:population options for population #116

Merged
merged 2 commits into from Jul 28, 2016

Conversation

Projects
None yet
3 participants
@avulfson17
Contributor

avulfson17 commented Jul 27, 2016

src/components/extractFields.js
@@ -10,11 +10,16 @@ function isUsCounty(base_record, qs_a2_alt) {
// this function favors gn:population when available, falling back to zs:pop10
// when available and > 0
+// the mz:population should be moved to the top when implemented

This comment has been minimized.

@trescube

trescube Jul 27, 2016

Contributor

If mz:population is moved to the top now then we won't have to change the code when it's supported in data.

@trescube

trescube Jul 27, 2016

Contributor

If mz:population is moved to the top now then we won't have to change the code when it's supported in data.

@trescube

This comment has been minimized.

Show comment
Hide comment
@trescube

trescube Jul 28, 2016

Contributor

:shipit:

Contributor

trescube commented Jul 28, 2016

:shipit:

@orangejulius

This comment has been minimized.

Show comment
Hide comment
Member

orangejulius commented Jul 28, 2016

LGTM

@orangejulius orangejulius merged commit 4a88df5 into master Jul 28, 2016

3 checks passed

approvals/lgtm this commit looks good
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@orangejulius orangejulius referenced this pull request in pelias/pelias Aug 1, 2016

Closed

2016/7/28 build acceptance-test issue tracker #390

6 of 6 tasks complete

@orangejulius orangejulius deleted the add-population-types branch Aug 5, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment