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

fix: fix search/getSuggestions when passing in both proximity and bbox #122

Merged
merged 1 commit into from
Mar 1, 2022

Conversation

thaddmt
Copy link
Member

@thaddmt thaddmt commented Feb 28, 2022

Description of changes

  • Amplify Geo/ Location Service would throw an error when passing in both a bounding box and a proximity parameter, fixed that by prioritizing bbox over proximity params

Issue #, if available

Description of how you validated changes

Checklist

  • PR description included
  • yarn test passes
  • Tests are [changed or added]
  • Relevant documentation is changed or added (and PR referenced)

@thaddmt thaddmt merged commit ce2205d into aws-amplify:main Mar 1, 2022
@github-actions
Copy link

github-actions bot commented Mar 9, 2022

🎉 This PR is included in version 1.3.0-ci-test.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions
Copy link

🎉 This PR is included in version 1.3.0-unstable.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

thaddmt added a commit that referenced this pull request Mar 17, 2022
* fix: change publish workflow if statement (#90) (#91)

* feat: add autoFit option to drawPoints (#94)

* feat: add autoFit option to drawPoints

* chore: update drawPoints unit tests

* chore: remove unused variable

* chore: add amplify-map.css file with various css configurations (#98)

* chore: add amplify-map.css file with various css configurations

* Update src/public/amplify-map.css

Co-authored-by: Amplifiyer <51211245+Amplifiyer@users.noreply.github.com>

Co-authored-by: Amplifiyer <51211245+Amplifiyer@users.noreply.github.com>

* prep v1.2.2 release (#96)

* fix: update circleci workflow to use main branch (#104)

* chore: prep v1.2.3 (#103)

* feat: Add stubs for auto suggest api (#106)

* feat: Add stubs for auto suggest api

* chore: update amplify geo version to include suggestions

* chore: update unit tests, fix suggestions response when undefined response

* feat: add flag for turning autocomplete off (#110)

* fix: update autocomplete logic (#111)

* chore: update amplify geocoder API docs (#113)

* chore: prep v1.2.4 (#112)

* chore: update node publish workflow version (#115)

* fix: temporarily disable autocomplete by default (#116)

* chore: prep v1.3.1 (#117)

* fix: remove regular dependency on maplibre-gl-geocoder (#119)

* fix: fix search/getSuggestions when passing in both proximity and bbox (#122)

* fix: update sourceId to be the sourceName passed into drawPoints (#123)

* fix: update sourceId to be the sourceName passed into drawPoints

* fix unit tests

* feat: allow for customizing markers with drawPoints (#124)

* chore: add codecov (#126)

* chore: Switch release process to use semantic-release (#127)

* chore: Switch release process to use semantic-release

* update node versions to use 16

* trying different v16 node

* update more node versions

* chore: added commitlint and husky

* chore: remove unnecessary github env var

* fix: add github token to publish workflow (#129)

* Change to use NODE_AUTH_TOKEN (#131)

* chore(ci): try switching to node_auth_token

* chore: remove npm_token

* chore: remove ci-test stuff

Co-authored-by: Amplifiyer <51211245+Amplifiyer@users.noreply.github.com>
Co-authored-by: Tré Ammatuna <16496746+TreTuna@users.noreply.github.com>
@github-actions
Copy link

🎉 This PR is included in version 1.3.0-geo.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

2 participants