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.

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

Fixes web interface #145

Closed
wants to merge 2 commits into from
Closed

Fixes web interface #145

wants to merge 2 commits into from

Conversation

cedarmora
Copy link

Fixes #136 web interface error and updates the mapping interface and README to reflect changes in the google maps API.

@ghost
Copy link

ghost commented Jun 30, 2019

It dont works

@cedarmora
Copy link
Author

PHPoenx, did you follow the instructions I added to the readme?

@ereyes01
Copy link

ereyes01 commented Jul 8, 2019

This worked for me, and the change is pretty simple. It would be nice if the API key could be passed in via a CLI arg, or an environment variable. But it worked for me as-is after following the instructions.

@cedarmora
Copy link
Author

cedarmora commented Jul 8, 2019 via email

@ZL1LAC
Copy link

ZL1LAC commented Jul 17, 2019

Any way to switch to OpenStreetMap?

@antirez
Copy link
Owner

antirez commented Jan 29, 2020

Would using open street map fix the issue? Or it would still require an API key? If OSM does not require any API key I could switch to their APIs.

@cedarmora
Copy link
Author

It looks like neither of the OSM tile providers require an API key:

https://switch2osm.org/using-tiles/getting-started-with-leaflet/

https://switch2osm.org/using-tiles/getting-started-with-openlayers/

@antirez
Copy link
Owner

antirez commented Jan 29, 2020

I tested it a few seconds, I think I really want to switch dump1090 to OSM. I'll do some work in the next days to see if it has all the features we need.

@antirez
Copy link
Owner

antirez commented Feb 3, 2020

Ported to OSM. Closing.

@antirez antirez closed this Feb 3, 2020
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.

Web interface needs an update
4 participants