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.

Sign up for GitHub

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

Jump to bottom

Update geocoder doc #8783

Merged

emackey merged 1 commit into master from OmarShehata-patch-1

Apr 23, 2020

Contributor

OmarShehata commented Apr 22, 2020

The geocoder doc was outdated. This updates it to state that the default geocoder in CesiumJS right now uses Cesium ion.


          Update geocoder doc

cesium-concierge commented Apr 22, 2020

Thanks for the pull request @OmarShehata!

✔️ Signed CLA found.
❔ CHANGES.md was not updated.
- If this change updates the public API in any way, please add a bullet point to CHANGES.md.
❔ Unit tests were not updated.
- Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

Cesium Viewer works.
Works in 2D/CV.
Works (or fails gracefully) in IE11.

emackey merged commit 70f8445 into master

emackey deleted the OmarShehata-patch-1 branch

April 23, 2020 01:10

OmarShehata mentioned this pull request

Geocoder summary documentation is incorrect #6870

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet