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

MM-62104 Fix board disappearing or move to default category when switching teams #44

Merged
merged 2 commits into from
Dec 12, 2024

Conversation

Rajat-Dabade
Copy link
Contributor

Summary

I was able to identify a way to reproduce this issue and pinpoint its root cause. This issue has been present for a long time, even before the pre-split.

Steps to Reproduce:

  • Create a category in the board sidebar.
  • Create a board within that category.
  • Observe that there are now two categories in the sidebar: the one you created and the default Boards category.
  • Switch to a different team.
  • Switch back to the team where you created the category.
  • Observe the issue: the board is either no longer in the created category or has been moved to the default Boards category.

Root Cause: This is a client-side issue caused by a this useEffect that executes before the Redux store is updated with the latest sidebar information. As a result, the /api/v2/teams/{teamID}/categories/{categoryId}/boards/{boardID} API is called with an empty categoryId. This causes the board to be incorrectly moved to the default Boards category.

Ticket Link

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

@Rajat-Dabade Rajat-Dabade added 2: Dev Review Requires review by a core committer 3: QA Review Requires review by a QA tester labels Dec 9, 2024
@Rajat-Dabade Rajat-Dabade self-assigned this Dec 9, 2024
@Rajat-Dabade Rajat-Dabade merged commit 5dbf729 into main Dec 12, 2024
6 checks passed
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 3: QA Review Requires review by a QA tester
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants