Skip to content

Fix route selection after delete#1425

Merged
patricvincit merged 1 commit into
mainfrom
31281/78763/delete-route-bug
May 25, 2026
Merged

Fix route selection after delete#1425
patricvincit merged 1 commit into
mainfrom
31281/78763/delete-route-bug

Conversation

@patricvincit

@patricvincit patricvincit commented May 12, 2026

Copy link
Copy Markdown
Contributor

When deleting an existing route, also clear route editor UI state:

  • Remove drawn route from map
  • Stop route editing mode
  • Clear selected route id

This prevents deleted routes from staying highlighted/selected on the map.

User story: 31281
Task: 78763


This change is Reviewable

@github-actions

github-actions Bot commented May 12, 2026

Copy link
Copy Markdown

Dependency Review

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

Scanned Files

None

@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 1 file and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on patricvincit).

When deleting an existing route, also clear route editor UI state:
- Stop route editing mode
- Clear selected route id

This prevents deleted routes from staying highlighted/selected on the map.

User story: 31281
Task: 78763
@patricvincit patricvincit force-pushed the 31281/78763/delete-route-bug branch from 32fd279 to 3ed4553 Compare May 18, 2026 11:52

@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 1 file and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on patricvincit).

@patricvincit patricvincit merged commit 2e3dcd2 into main May 25, 2026
24 checks passed
@patricvincit patricvincit deleted the 31281/78763/delete-route-bug branch May 25, 2026 09:10
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