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

UI: Add stop filtering and prioritization for search results #641

Merged
merged 8 commits into from
Feb 28, 2025

Conversation

ksharma-xyz
Copy link
Owner

@ksharma-xyz ksharma-xyz commented Feb 26, 2025

Stop Search Prioritization and Filtering Improvements

  • Introduced prioritiseStops function to sort stops based on transport mode priority order
  • Moved stop filtering logic from SQL to ViewModel layer for better control
  • Added unit tests for product class filtering functionality
  • Implemented high priority stops sorting with predefined list:
    • Town Hall (200060)
    • Central (200070)
    • Wynyard (200080)
    • Circular Quay (206010)
    • Bondi Junction (2150106)
    • Chatswood (200017)
    • Parramatta (200039)
    • Blacktown (201016)
    • Liverpool (201039)
    • Penrith (201080)
    • St James (200066)
    • Museum (200030)
    • Martin Place (200046)
    • Kings Cross (200050)
  • Results are now limited to 50 stops after sorting and filtering

@ksharma-xyz ksharma-xyz changed the title prioritiseStops based on priority order UI: Add stop filtering and prioritization for search results Feb 26, 2025
@ksharma-xyz ksharma-xyz marked this pull request as ready for review February 26, 2025 21:24
@ksharma-xyz ksharma-xyz force-pushed the 02-23-match_stopresults_from_db_instead_of_api_call branch from 9a245d9 to d1ff69f Compare February 27, 2025 13:22
@ksharma-xyz ksharma-xyz force-pushed the 02-24-prioritisestops_based_on_priority_order branch from 87aa564 to 756f2c7 Compare February 27, 2025 13:22
@ksharma-xyz ksharma-xyz force-pushed the 02-23-match_stopresults_from_db_instead_of_api_call branch from d1ff69f to 007126c Compare February 27, 2025 13:33
@ksharma-xyz ksharma-xyz force-pushed the 02-24-prioritisestops_based_on_priority_order branch 2 times, most recently from 075c995 to 86bc7e1 Compare February 28, 2025 06:35
@ksharma-xyz ksharma-xyz force-pushed the 02-23-match_stopresults_from_db_instead_of_api_call branch from 007126c to fe6f009 Compare February 28, 2025 06:35
@ksharma-xyz ksharma-xyz force-pushed the 02-23-match_stopresults_from_db_instead_of_api_call branch from fe6f009 to d254c91 Compare February 28, 2025 06:51
@ksharma-xyz ksharma-xyz force-pushed the 02-24-prioritisestops_based_on_priority_order branch from 86bc7e1 to 51a1318 Compare February 28, 2025 06:52
@ksharma-xyz ksharma-xyz force-pushed the 02-23-match_stopresults_from_db_instead_of_api_call branch 2 times, most recently from 6948c2f to ef9fe79 Compare February 28, 2025 07:03
@ksharma-xyz ksharma-xyz force-pushed the 02-24-prioritisestops_based_on_priority_order branch 2 times, most recently from ab8bb33 to f821ff1 Compare February 28, 2025 08:02
@ksharma-xyz ksharma-xyz changed the base branch from 02-23-match_stopresults_from_db_instead_of_api_call to graphite-base/641 February 28, 2025 08:14
@ksharma-xyz ksharma-xyz force-pushed the 02-24-prioritisestops_based_on_priority_order branch from f821ff1 to ffa3ae3 Compare February 28, 2025 08:15
@ksharma-xyz ksharma-xyz changed the base branch from graphite-base/641 to main February 28, 2025 08:15
@ksharma-xyz ksharma-xyz force-pushed the 02-24-prioritisestops_based_on_priority_order branch from ffa3ae3 to 81f22de Compare February 28, 2025 08:15
@ksharma-xyz ksharma-xyz merged commit f053471 into main Feb 28, 2025
2 checks passed
Copy link
Owner Author

Merge activity

  • Feb 28, 3:32 AM EST: A user merged this pull request with Graphite.

@ksharma-xyz ksharma-xyz deleted the 02-24-prioritisestops_based_on_priority_order branch February 28, 2025 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant