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: Adhere limitations in tariff zone selector #4917

Merged
merged 5 commits into from
Jan 14, 2025

Conversation

gorandalum
Copy link
Member

@gorandalum gorandalum commented Jan 10, 2025

When there is limitation on multiple tariff zones for a product, the
user is able to navigate to the tariff zone selection map. But in
that map all zones were actually shown, not just the one applicable
by the limitation.

Acceptance criteria

  • Only selectable tariff zones should be shown and pressable in the map
  • Only selectable tariff zones should be listed in the empty search view
  • Only stop places for selectable tariff zones should be returned searching

How to test:

  • Either by proxy or by changing Firestore create a list of two or more (but not all) tariff zones in the preassignedFareProduct.limitations.tariffZoneRefs

This means that we no longer need to pass around multiple purchase
related fields when navigating between purchase screens.

I wasn't able to split this up more than this since as soon as the
route param was changed all of these places needed to be fixed.
When there is limitation on multiple tariff zones for a product, the
user is able to navigate to the tariff zone selection map. But in
that map all zones were actually shown, not just the one applicable
by the limitation.
@gorandalum gorandalum self-assigned this Jan 10, 2025
Base automatically changed from dalum/purchase_selection_in_route_params to master January 14, 2025 12:38
@gorandalum gorandalum merged commit 5dd91ab into master Jan 14, 2025
4 checks passed
@gorandalum gorandalum deleted the dalum/only_show_selectable_tariff_zones_in_map branch January 14, 2025 14:59
@tormoseng
Copy link
Member

✅ By editing the referenceData.

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.

3 participants