Skip to content

Expand checks for popup creation to handle undefined in addition to null#1

Open
rebfrank wants to merge 2 commits intojs_testsfrom
data_not_avail_checks
Open

Expand checks for popup creation to handle undefined in addition to null#1
rebfrank wants to merge 2 commits intojs_testsfrom
data_not_avail_checks

Conversation

@rebfrank
Copy link
Owner

  • Perhaps the Python data will never have a missing field, but if it does, this change will handle
    it. (I detected this in generating "error" test case data; I have not seen any evidence of this affecting the real site data).
  • This modifies the error checking when generating the popup to make sure all data points used are
    not null or undefined.
  • The main change is in index.js, but the relevant test case is "popup of district missing data shows data not available error"

- Perhaps the Python data will never have a missing field, but if it does, this change will handle
  it.
- This modifies the error checking when generating the popup to make sure all data points used are
  not null or undefined.
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.

1 participant