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.

Sign up for GitHub

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

Jump to bottom

Add sumlevels 252 and 254 to SUMLEV_NAMES #52

Merged
merged 1 commit into from
Aug 23, 2016
Merged

Add sumlevels 252 and 254 to SUMLEV_NAMES #52

merged 1 commit into from
Aug 23, 2016

Conversation

tuchandra
Copy link
Member

@tuchandra tuchandra commented Aug 23, 2016 •
edited
Loading

See censusreporter/censusreporter#34. In order for these geographies to be handled correctly, three things are necessary:

  • The geographies have to appear in tiger2014.census_name_lookup with information (namely, geoids) that do not contradict the ACS geoheader table.
  • The API has to return these geographies.
  • Search results and autocomplete have to include these geographies.

This handles the second, by adding sumlevels 252 and 254 to the list of allowed sumlevels. This allows the sumlevs parameter in the API requests to include 252 and 254 as valid entries.

@tuchandra
Add sumlevels 252 and 254 to SUMLEV_NAMES
2e39135 
Add sumlevels 252 and 254 to the list of valid sumlevels. This is
related to censusreporter/censusreporter#34 with proper handling
of sumlevels 250, 252, and 254.
@tuchandra tuchandra mentioned this pull request Aug 23, 2016
Closed
@JoeGermuska JoeGermuska merged commit 4f2d40e into master Aug 23, 2016
@JoeGermuska JoeGermuska deleted the fix-sumlevel-250 branch August 23, 2016 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tuchandra @JoeGermuska