fix(scm-messaging-integration-ui): Enable picking from multiple integ… - #122092
fix(scm-messaging-integration-ui): Enable picking from multiple integ…#122092Abdkhan14 wants to merge 6 commits into
Conversation
github-actions
Bot
added
the
Scope: Frontend
📊 Type Coverage Diff✅ no issues found |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit c619998. Configure here.
| setSelectedIntegrationId(undefined); | ||
| setChannel(undefined); | ||
| } | ||
| }, [eligibleIntegrations, selectedIntegrationId]); |
There was a problem hiding this comment.
Stale channel after default workspace removal
Medium Severity
The workspace-removal effect only clears the channel when an explicit selectedIntegrationId disappears. When the effective workspace comes from defaultIntegration (saved or first, with selectedIntegrationId still undefined) and that workspace drops out of eligibleIntegrations, the derived workspace falls back but the channel is left in place. Saving can then pair the old channel with a different integrationId.
Reviewed by Cursor Bugbot for commit c619998. Configure here.
There was a problem hiding this comment.
This is valid taking a look
1 participant
...rations.