Skip to content

72784/edit tram stop on map#1398

Merged
PasiVuohijoki merged 4 commits into
mainfrom
72784/edit-tram-stop-on-map
Mar 31, 2026
Merged

72784/edit tram stop on map#1398
PasiVuohijoki merged 4 commits into
mainfrom
72784/edit-tram-stop-on-map

Conversation

@PasiVuohijoki

@PasiVuohijoki PasiVuohijoki commented Mar 30, 2026

Copy link
Copy Markdown
Contributor

Lots of small fixes for how the filters are enabled and stops & stop areas showed when moving into map view from different places.

Added the transportation mode info into edit stop on map dialog.


This change is Reviewable

@github-actions

github-actions Bot commented Mar 30, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 7478f77.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@PasiVuohijoki PasiVuohijoki force-pushed the 72784/edit-tram-stop-on-map branch 3 times, most recently from 3f34bab to 67d6402 Compare March 30, 2026 13:39

@Huulivoide Huulivoide left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@Huulivoide reviewed 15 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on PasiVuohijoki).

@PasiVuohijoki PasiVuohijoki force-pushed the 72784/edit-tram-stop-on-map branch from 67d6402 to dce596d Compare March 31, 2026 09:31
- Use parseVehicleMode when mapping stop data and filtering by vehicle mode
  to handle uppercase values (e.g. "TRAM") from the stops database.
When navigating to the map via "show on map" from stop list,
stop details or stop area views, the map filters were not adjusted
to match the stop's actual vehicle mode or priority.
@PasiVuohijoki PasiVuohijoki force-pushed the 72784/edit-tram-stop-on-map branch from dce596d to 7478f77 Compare March 31, 2026 12:37
@PasiVuohijoki PasiVuohijoki merged commit 1eedac6 into main Mar 31, 2026
23 of 24 checks passed
@PasiVuohijoki PasiVuohijoki deleted the 72784/edit-tram-stop-on-map branch March 31, 2026 12:59
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.

2 participants