feat(integrations): prompt user to import after provider connection#2383
feat(integrations): prompt user to import after provider connection#2383
Conversation
…ection Resolves ENG-114 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
PR SummaryLow Risk Overview This prompt is triggered in both the non-OAuth connect-dialog success path and the OAuth callback path (in addition to the existing success toast). Unit tests were updated/added to mock Written by Cursor Bugbot for commit 457e5a0. This will update automatically on new commits. Configure here. |
![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.
| onClick: () => router.push(`/${orgId}/people/all`), | ||
| }, | ||
| }); | ||
| } |
There was a problem hiding this comment.
Duplicated employee sync toast logic across two paths
Low Severity
The employee sync import prompt toast.info call is identically duplicated in handleConnectDialogSuccess and the OAuth callback useEffect. Both share the same message template, description, duration, and action configuration. If the toast content, duration, or navigation target changes, both sites need independent updates, risking inconsistency. Extracting a small helper like showEmployeeSyncPrompt(providerName) would keep the logic in one place.
Additional Locations (1)
2 participants
Summary
Test plan
Resolves ENG-114
🤖 Generated with Claude Code