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

[BUILD] Maps service config #308

Merged
merged 1 commit into from
Apr 24, 2021

Conversation

kavilla
Copy link
Member

@kavilla kavilla commented Apr 22, 2021

Description

This doesn't fix the maps out of the box but fixes the issues that prevented users from settings the maps service in the config.

Which can be set by adding the following to opensearch_dashboards.yml:

map.tilemap.url: "https://tiles.maps.search-services.aws.a2z.com/tiles/{z}/{x}/{y}.png"
map.includeOpenSearchMapsService: false

Signed-off-by: Kawika Avilla [email protected]

Issues Resolved

Partial: #221

Check List

  • New functionality includes testing.
    • All tests pass (besides functional tests)
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff

This doesn't fix the maps out of the box but fixes the
issues that prevented users from settings the maps service
in the config.

Which can be set by adding:

map.tilemap.url: "https://tiles.maps.search-services.aws.a2z.com/tiles/{z}/{x}/{y}.png"
map.includeOpenSearchMapsService: false

Signed-off-by: Kawika Avilla <[email protected]>
@kavilla kavilla added bug Something isn't working >Fork rename labels Apr 22, 2021
@kavilla kavilla added this to the Beta release milestone Apr 22, 2021
@odfe-release-bot
Copy link

✅   DCO Check Passed 97cb55a

Copy link
Contributor

@mihirsoni mihirsoni left a comment

Choose a reason for hiding this comment

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

@kavilla Thanks for your changes, as discussed let's modify default client to use the latest Maps client.

Copy link
Contributor

@boktorbb boktorbb left a comment

Choose a reason for hiding this comment

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

Looks good

@kavilla
Copy link
Member Author

kavilla commented Apr 23, 2021

@kavilla Thanks for your changes, as discussed let's modify default client to use the latest Maps client.

Can that go in a subsequent PR? This fixes current functionality that is broken.

Copy link
Contributor

@mihirsoni mihirsoni left a comment

Choose a reason for hiding this comment

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

LGTM !!

@mihirsoni mihirsoni merged commit 5e08fe3 into opensearch-project:main Apr 24, 2021
@kavilla kavilla deleted the kavilla/map-errors branch April 27, 2021 19:48
kavilla added a commit that referenced this pull request May 21, 2021
This doesn't fix the maps out of the box but fixes the
issues that prevented users from settings the maps service
in the config.

Which can be set by adding:

map.tilemap.url: "https://tiles.maps.search-services.aws.a2z.com/tiles/{z}/{x}/{y}.png"
map.includeOpenSearchMapsService: false

Signed-off-by: Kawika Avilla <[email protected]>
SuZhou-Joe added a commit to SuZhou-Joe/OpenSearch-Dashboards that referenced this pull request Mar 25, 2024
…oject#6094) (opensearch-project#308)

* feat: add comment



* feat: update unit test



* feat: add CHANGELOG



* feat: optimize comment



* feat: optimize comment



* feat: optimize code



* feat: optimize code



---------

Signed-off-by: SuZhou-Joe <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working rename
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants