-
Notifications
You must be signed in to change notification settings - Fork 27
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
Open Street Maps jako mapa #12
base: develop
Are you sure you want to change the base?
Conversation
zaraz zcherry-pickuje tutaj tylko ostatni commit i odrzucę zmiany z dockerem |
fa53ec9
to
2d9aff6
Compare
@adlorenz, dla mnie zmiana gotowa do wdrożenia. Merytoryczna zmiana to JS więc to można łatwo sobie przetestować lokalnie "dopisując" w DevToolsach te linijki albo wskazując w ogóle podmianę na plik lokalny. np. https://developer.chrome.com/docs/devtools/overrides?hl=pl |
Zrobiłem tę podmiankę lokalnie w Chrome, korzystając z DevTools. Wygląda świetnie. :) Pytanie skąd pobierana jest mapa i jak wygląda licencja na wykorzystanie OSM w tym projekcie? |
PS. Wdrożyłem hot-fix także na produkcji. :) Dzięki za kontrybucję! Podpowiedz proszę jak wygląda kwestia licencji, ew. gdzie szukać info na ten temat. Thx. |
Superancko @adlorenz! Dzięki! Licencja wygląda na to że powinniśmy jeszcze wrzucić info w rogu że to "autorzy OpenStreetMap" No i pewnie warto wywalić logo google o ile się uda :D |
ja też dziekuję :) |
wygląda na to, że działa całkiem ok, akurat w tym samym czasie był artykuł na stronie https://www.dobreprogramy.pl/mapa-zasiegu-5g-i-lte-gdzie-jest-sygnal-komorkowy,6991999821507520a który linkował do mapy, więc perfekcyjnie się zgrało 😄 |
Na podstawie wspaniałego komentarza:
#4 (comment)
Dzięki @pmatlak1
closes #4
closes #1
closes #5