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

hotfix: no need to redirect when removing tab. router handles this fine #17780

Merged

Conversation

nielslyngsoe
Copy link
Member

@nielslyngsoe nielslyngsoe commented Dec 11, 2024

Fixes issue when removing tab from a document type, but it was only a problem when user was on another tab than the one to be deleted.

Fixes #17771

see
https://umbraco.slack.com/archives/C027U315T54/p1733894530378609?thread_ts=1733894482.267889&cid=C027U315T54

Notice, the problem seemed to be a re-direct and this was not needed anymore, the router seems to handle this fine by its own redirects.

Prerequisites

  • I have added steps to test this contribution in the description below

If there's an existing issue for this PR then this fixes

Description

@nielslyngsoe nielslyngsoe changed the title no need to redirect when removing tab. router handles this fine hotfix: no need to redirect when removing tab. router handles this fine Dec 11, 2024
Copy link
Contributor

@madsrasmussen madsrasmussen left a comment

Choose a reason for hiding this comment

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

Changes look good, and it is now possible to delete tabs.

@iOvergaard iOvergaard merged commit 5f291ba into release/15.1 Dec 11, 2024
30 checks passed
@iOvergaard iOvergaard deleted the v15/hotfix/do-not-redirect-when-removing-tab branch December 11, 2024 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants