Skip to content

[MM-67519] Remove clean path name#3699

Open
devinbinnie wants to merge 1 commit intomasterfrom
MM-67519
Open

[MM-67519] Remove clean path name#3699
devinbinnie wants to merge 1 commit intomasterfrom
MM-67519

Conversation

@devinbinnie
Copy link
Copy Markdown
Member

Summary

In cases where a team name is the same as the subpath name in a URL (eg. https://foo.bar/foo/foo/test, where foo is the team name and test is the channel name), the Desktop App browser history logic gets confused and will treat foo as the channel name and remove the second foo, resulting in a Team not found error.

We added this because Boards uses a different browser history module, and it does not account for the basename when setting up the navigation. However, we will be changing that over, so this PR will remove that extra path cleaning logic altogether.

Ticket Link

https://mattermost.atlassian.net/browse/MM-67519

Fixed an issue where subpath servers might show a "Team not found" error in some cases

@devinbinnie devinbinnie requested review from a team and agarciamontoro and removed request for a team February 17, 2026 19:11
@devinbinnie devinbinnie added 2: Dev Review Requires review by a core committer Do Not Merge Should not be merged until this label is removed labels Feb 17, 2026
@devinbinnie devinbinnie added the Build Apps for PR Builds signed builds for testing label Feb 17, 2026
@devinbinnie
Copy link
Copy Markdown
Member Author

@Rajat-Dabade this has the sample build for testing the Boards history module

We should not merge this until we've decided when we should introduce a breaking change for previous Boards versions.

@yasserfaraazkhan yasserfaraazkhan added the E2E/Run Run Desktop E2E Tests label Apr 16, 2026
@mm-cloud-bot
Copy link
Copy Markdown

❌ E2E Test Setup Failed

Failed to create E2E test instances: failed to create installation: failed with status code 409

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2: Dev Review Requires review by a core committer Build Apps for PR Builds signed builds for testing Do Not Merge Should not be merged until this label is removed E2E/Run Run Desktop E2E Tests release-note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants