updates
This is looking great!
A couple points of feedback & requests for updates on the Leaflet side of the world:
- [x] Leaflet.markercluster's
zoomToBoundsOnClickisn't working, can you debug and get it to work. - [ ] I'd like it so each icon gets a popup when you click it saying the location and the GitHub user who added it. Can you please do so? You'll need to edit src/locations.js to make that work.
- [ ] Because Leaflet doesn't use Google Maps tiles, we need a new geocoding engine, can you find one and let me know what we need to get it working?
Regarding automerging, I think a better implementation would be to implement a test framework in this repository. The test framework should test the merge, see that only one line was changed and then mark that test passing.
It should then check the GitHub PR to see if there's a react on it by someone other than the author. If there is that test should pass.
If both tests pass it should then have a separate action (which should already exist) that automeges when there are passed tests.
- [ ] Because Leaflet doesn't use Google Maps tiles, we need a new geocoding engine, can you find one and let me know what we need to get it working?
Potential API: https://docs.mapbox.com/api/search/geocoding/#endpoints -- may cost money? Would have to contact Sales team, but perhaps they'd be willing to work something out?