Closed carloscdias closed 6 years ago
Looks great!
The new name
property appears in the sample JSON but not in the full data. @mikefab, would it be possible to add the name field to this data so that this code works with test or "real" data? (Or if there's a way I can do it, just let me know). Thank you!
After speaking with Mike and David yesterday, it sounds like David's work on the aggregation story will probably result in us getting more "real" data. Carlos noted that any properties other than color
will be displayed in the popup, so we probably can live without the name
property for now.
Anyway, looks great - merging!
Summary
This PR adds a simple handler for click on schools, similar to the one presented here. It can be used as an example to develop new and more complex handlers.