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

feat: autosuggest place id support #173

Merged
merged 7 commits into from Oct 17, 2022

Conversation

sreeramsama
Copy link
Contributor

@sreeramsama sreeramsama commented Sep 26, 2022

Description of changes

Add getSuggestions and searchByPlaceId APIs with unit tests

Issue #, if available

Description of how you validated changes

Manual testing with test data on UI component and unit tests.

Checklist

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

thaddmt
thaddmt previously approved these changes Sep 26, 2022
@codecov-commenter
Copy link

codecov-commenter commented Oct 14, 2022

Codecov Report

Merging #173 (b986d64) into main (5e2ff6c) will increase coverage by 0.72%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #173      +/-   ##
==========================================
+ Coverage   42.94%   43.67%   +0.72%     
==========================================
  Files          16       16              
  Lines        1248     1264      +16     
  Branches      169      170       +1     
==========================================
+ Hits          536      552      +16     
  Misses        712      712              
Impacted Files Coverage Δ
src/AmplifyMapLibreGeocoder.ts 92.30% <100.00%> (+2.51%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

thaddmt
thaddmt previously approved these changes Oct 17, 2022
Copy link
Contributor

@thaddmt thaddmt left a comment

Choose a reason for hiding this comment

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

Have you tried testing this change with the new geocoder changes? Otherwise LGTM

@thaddmt
Copy link
Contributor

thaddmt commented Oct 17, 2022

Also I believe we may need to preface this PR title with feat: in order to trigger a change from semver when we do the release because when we do a squash and merge it will just use this PR's title as the commit title

@sreeramsama sreeramsama changed the title Autosuggest place id support feat: autosuggest place id support Oct 17, 2022
@sreeramsama sreeramsama merged commit 491ee04 into aws-amplify:main Oct 17, 2022
@sreeramsama sreeramsama deleted the autosuggest-placeId-support branch October 17, 2022 20:04
@github-actions
Copy link

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

The release is available on:

Your semantic-release bot 📦🚀

sreeramsama added a commit that referenced this pull request Oct 19, 2022
* feat: add Geocoder APIs with test data - getSuggestions, searchByPlaceId

* chore: update getSuggestions and searchByPlaceId APIs with unit tests and proper return types

* fix: fix unit test and check for undefined return value

* chore: use newer version of maplibre-gl-geocoder

* chore: use latest @aws-amplify/geo

* chore: use latest @aws-amplify/geo
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.

None yet

3 participants