Skip to content

fix: satellite imagery is broken in GeoMap and GlobalMap#46

Merged
tarekio merged 13 commits intomainfrom
BYNT-1327-Satellite-imagery-is-broken-in-GeoMap-and-GlobalMap
Jan 16, 2025
Merged

fix: satellite imagery is broken in GeoMap and GlobalMap#46
tarekio merged 13 commits intomainfrom
BYNT-1327-Satellite-imagery-is-broken-in-GeoMap-and-GlobalMap

Conversation

@apodacaduron
Copy link
Contributor

Jira Issue

  1. 1327

Description

Satellite imagery is supposed to be enabled if a google maps api is added to the setting. A button will appear which allows the user to switch between OSM and Google. However that button doesn’t appear now. Google Maps js seems to be loaded correctly so the issue could be in the two components themselves, possible introduced during v3 migration.

Checklist

  • Tests added/updated
  • Documentation updated (if needed)
  • New strings prepared for translations

API Changes (if applicable)

  • Permissions checked
  • Endpoint tests added

Additional Notes

[Any other relevant information]

@apodacaduron apodacaduron requested a review from tarekio January 2, 2025 23:58
@apodacaduron apodacaduron self-assigned this Jan 2, 2025
Copy link
Contributor

@tarekio tarekio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is good so far but we need to add proper attribution for the satellite imagery: https://about.google/brand-resource-center/products-and-services/geo-guidelines/#required-attribution

@tarekio tarekio changed the title Satellite imagery is broken in GeoMap and GlobalMap fix: satellite imagery is broken in GeoMap and GlobalMap Jan 14, 2025
Copy link
Contributor

@tarekio tarekio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@level09 please review and approve.

@tarekio tarekio requested a review from level09 January 14, 2025 19:17
Copy link
Collaborator

@level09 level09 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @apodacaduron just a small request. can we convert the coding style to use options api ?

@apodacaduron
Copy link
Contributor Author

@level09 updated back to options api should be working now

Copy link
Collaborator

@level09 level09 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@tarekio tarekio merged commit da65f44 into main Jan 16, 2025
@tarekio tarekio deleted the BYNT-1327-Satellite-imagery-is-broken-in-GeoMap-and-GlobalMap branch January 16, 2025 21:10
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.

3 participants