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

Geojson missing mcd and county names #27

Closed
jstrangfeld opened this issue Mar 5, 2018 · 3 comments · Fixed by #42
Closed

Geojson missing mcd and county names #27

jstrangfeld opened this issue Mar 5, 2018 · 3 comments · Fixed by #42

Comments

@jstrangfeld
Copy link
Member

Sean, please join the municipality name (MUN1) and county name (CNTY_NAME) to the dataset.

@seandoesgis

https://github.com/crvanpollard/IPD_V2/blob/8b09891714d22b935b5d9cf80467328541308049/lib/map.js#L274
@rbeatty1 rbeatty1 assigned rbeatty1 and unassigned rbeatty1 Mar 7, 2018
@rbeatty1
Copy link
Contributor

rbeatty1 commented Mar 7, 2018

Going to remove these for the time being. @seandoesgis if you could add the municipality and county fields to the dataset that would be great, but isn't mission critical at this point. Drop me a note when you do and I'll add this back in.

@crvanpollard crvanpollard mentioned this issue Mar 12, 2018
Closed
@crvanpollard crvanpollard reopened this Mar 12, 2018
@crvanpollard
Copy link
Member

Update IPD dataset on DVRPC Open Data Portal to include MCD and County names. This is important information for users accessing and review IPD scores and analysis for their desired township or borough.

@crvanpollard
Copy link
Member

IPD 2016 feature class with MCD and CNTY values can be found here: U:_OngoingProjects\EJ\2015_IPD\NewMethodology_WebMap\IPD_2016_geojson.shp
attribute fields that need to be included: MUN1, MUN2, MUN3, CNTY_NAME

crvanpollard added a commit that referenced this issue Mar 14, 2018
@crvanpollard
add mcd and cnty name
27e3f83 
Closes #27
- also fixes Upper Case vs Lower case attribute values. attribute
headings changed in GeoJson API service
@crvanpollard crvanpollard mentioned this issue Mar 14, 2018
Merged
crvanpollard added a commit that referenced this issue Mar 14, 2018
@crvanpollard
Merge pull request #42 from crvanpollard/pollard_wokring
ac95666 
closes #27 
add mcd and cnty names (also fixes an issue w/ UPPER vs Lower case that occured with ESRI Open Data GeoJson API)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Pollard wokring dvrpc/ipd-viewer
3 participants
@crvanpollard @jstrangfeld @rbeatty1