Fix GeoJSON syntax - Githubissues

mozillascience / studyGroupLessons

One-hour introductory lessons on ideas and tools in coding and data wrangling for research.

MIT License

114 stars 37 forks source link

Fix GeoJSON syntax #60

Closed abought closed 8 years ago

abought commented 8 years ago

Purpose

Fix syntax error in GeoJSON file that prevented the "where we are" map from loading/previewing correctly. (the page would infinitely appear to load when previewing the file from github)

Summary of changes

Fixed missing bracket that prevented map from loading
Made indentation more consistent (IDE auto-reformat)

bkatiemills commented 8 years ago

Thanks for looking into this! But there's still a bracket mismatch - the { at line 276 needs a friend.

abought commented 8 years ago

there's still a bracket mismatch - the { at line 276 needs a friend.

I have to confess I'm not seeing the reported mismatch when using the new version on this branch... odd.

I tried the following things:

Counted braces to check for mismatch (67 opening brackets/ 67 closing brackets)
Confirmed that file serializes to and from valid JSON, and previewing it now gives a map
Ran a copy through jshint and jsonlint syntax checkers (old file version failed with errors, new one passes)

bkatiemills commented 8 years ago

Whoops, right you are - I was looking at the wrong branch. Thanks for this!