-
-
Notifications
You must be signed in to change notification settings - Fork 285
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: 5568 - optimized search for price locations #5587
feat: 5568 - optimized search for price locations #5587
Conversation
Impacted files: * `app_en.arb`: added a label for "location broader search" * `app_fr.arb`: added a label for "location broader search" * `dao_osm_location.dart`: added columns osmKey and osmValue * `local_database.dart`: upgraded the version in order to add columns to location table * `location_list_supplier.dart`: added optional parameters for optimized search; added fields osmKey and osmValue * `location_query_model.dart`: now we use 2 suppliers - optimized and broader * `location_query_page.dart`: now display optimized results first, then broader results after a button click * `osm_location.dart`: added fields osmKey and osmValue
cc @raphodn |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #5587 +/- ##
==========================================
- Coverage 9.54% 6.94% -2.60%
==========================================
Files 325 402 +77
Lines 16411 21238 +4827
==========================================
- Hits 1567 1476 -91
- Misses 14844 19762 +4918 ☔ View full report in Codecov by Sentry. |
Thank you @teolemon @g123k for your reviews! @raphael0202 @raphodn As discussed less than a month ago, the location search needed to be improved.
I don't know if it's the method you're currently implementing for the web app. Feel free to make suggestions! |
I think it's amenity/shops only on the web app, but to be confirmed! |
What
shop
andamenity
), and to let the user enlarge the search if needed.Screenshots
Fixes bug(s)
Impacted files
app_en.arb
: added a label for "location broader search"app_fr.arb
: added a label for "location broader search"dao_osm_location.dart
: added columns osmKey and osmValuelocal_database.dart
: upgraded the version in order to add columns to location tablelocation_list_supplier.dart
: added optional parameters for optimized search; added fields osmKey and osmValuelocation_query_model.dart
: now we use 2 suppliers - optimized and broaderlocation_query_page.dart
: now display optimized results first, then broader results after a button clickosm_location.dart
: added fields osmKey and osmValue