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.

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

Fix/web 2228/search accuracy correction #3

Open
wants to merge 3 commits into
base: default-options
Choose a base branch
from
Open

Fix/web 2228/search accuracy correction #3

wants to merge 3 commits into from

Conversation

xavibonell-badi
Copy link

📌 References

🎩 What is the goal?

Modify the module's behaviour so it always searches through the Google geocode API

📝 How is it being implemented?

  • Modifying the method where fetches detailed information about the suggested list item.

🎨 UI changes?

  • Nope, the UI remains as beautiful as it was before!
  • Yeap, here you have some screenshots-

@xavibonell-badi xavibonell-badi self-assigned this Feb 26, 2020
@xavibonell-badi xavibonell-badi changed the base branch from master to default-options February 26, 2020 14:57
@xavibonell-badi xavibonell-badi changed the base branch from default-options to master February 26, 2020 15:06
@xavibonell-badi xavibonell-badi force-pushed the fix/WEB-2228/search-accuracy-correction branch from 569fa26 to 7e3f714 Compare February 26, 2020 15:13
@xavibonell-badi xavibonell-badi changed the base branch from master to default-options February 26, 2020 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adriamarzo adriamarzo adriamarzo approved these changes

Assignees

@xavibonell-badi xavibonell-badi

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@xavibonell-badi @adriamarzo