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

Add satelite map #111

Closed
Petrprogs opened this issue Jun 23, 2023 · 11 comments · Fixed by #116
Closed

Add satelite map #111

Petrprogs opened this issue Jun 23, 2023 · 11 comments · Fixed by #116
Assignees
Labels

Comments

@Petrprogs
Copy link
Contributor

No description provided.

@Feodor0090
Copy link
Contributor

Feodor0090 commented Jun 23, 2023

This needs further JS API / web app reverse engineering to understand what options core-renderer wants from us to return non-scheme tiles.

@Petrprogs
Copy link
Contributor Author

I already found satelite tiles url.
For example - for getting scheme tiles we use url like that: https://core-renderer-tiles.maps.yandex.net/tiles?l=map&lang=ru_RU&x=10104&y=5148&z=14
So for getting satelite tiles we will use url like that: https://core-sat.maps.yandex.net/tiles?l=sat&lang=ru_RU&x=10104&y=5148&z=14

Url for satelite tiles is: core-sat.maps.yandex.net
All params, instead of "l" key value are the same.

@Feodor0090
Copy link
Contributor

So tasklist is:

  • Delete allowDownload flag and replace it with integer field (0=offline, 1=scheme, 2=sattelite), update settings screen
  • Modify getUrl to use sattelite url.
  • Add hotkey to switch scheme/sat (0 key is still free)

You can implement all this and open PR if you want

@Feodor0090
Copy link
Contributor

@Petrprogs делаем?

Если это сделают за нас мне будет только проще

А ты если нечего делать сделай настройку прокси.

@Feodor0090
Copy link
Contributor

И на тебе (@shinovon) ещё висит на сервере обратывать флаги телеметрии (чекни апдейт тред, там новый GET параметр шлётся)

@Feodor0090
Copy link
Contributor

И определитесь на каком языке мы в этом репе общаемся, кстати

@Petrprogs
Copy link
Contributor Author

@Petrprogs делаем?

I think... Yes... But this will be not very fast :)

@shinovon
Copy link
Collaborator

И определитесь на каком языке мы в этом репе общаемся, кстати

я это случайно из тг отправил, лол

@shinovon
Copy link
Collaborator

И на тебе (@shinovon) ещё висит на сервере обратывать флаги телеметрии (чекни апдейт тред, там новый GET параметр шлётся)

давно сделано, читай чат хоть раз в год

@Petrprogs
Copy link
Contributor Author

И определитесь на каком языке мы в этом репе общаемся, кстати

я это случайно из тг отправил, лол

Can you pls write in Eng?

@Feodor0090
Copy link
Contributor

@shinovon мне в падлу

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants