Skip to content

feat(integrations): prompt user to import after provider connection#2383

Merged
Marfuen merged 3 commits intomainfrom
eng-114-prompt-import-after-google-workspace
Apr 2, 2026
Merged

feat(integrations): prompt user to import after provider connection#2383
Marfuen merged 3 commits intomainfrom
eng-114-prompt-import-after-google-workspace

Conversation

@tofikwest
Copy link
Copy Markdown
Contributor

Summary

  • After connecting a sync-capable provider (Google Workspace, Rippling, JumpCloud, Ramp), shows a toast prompting users to import their team members from People > All
  • Toast has a 15-second duration with a "Go to People" action button that navigates directly
  • Covers both OAuth callback and non-OAuth (API key) connection paths
  • Generalized for all sync-capable providers, not just Google Workspace

Test plan

  • Unit tests added (2 tests: positive for Google Workspace, negative for non-sync providers)
  • Connect Google Workspace — verify import prompt toast appears
  • Connect a non-sync provider (e.g., GitHub) — verify no import prompt
  • Click "Go to People" button — verify navigation to People > All

Resolves ENG-114

🤖 Generated with Claude Code

…ection

Resolves ENG-114

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@linear
Copy link
Copy Markdown

linear bot commented Mar 30, 2026

@cursor
Copy link
Copy Markdown

cursor bot commented Mar 30, 2026

PR Summary

Low Risk
Low risk UI-only change that adds a post-connection toast and navigation action; main risk is unintended prompting or routing on provider connect/callback paths.

Overview
Adds an employee import prompt after connecting certain sync-capable integrations (google-workspace, rippling, jumpcloud, ramp). After a successful connection—both via OAuth callback and via the non-OAuth connect dialog—the UI now shows a toast.info with a 15s duration and a “Go to People” action that routes to /${orgId}/people/all.

Updates PlatformIntegrations tests to mock useRouter and toast.info, and adds coverage to assert the prompt appears for Google Workspace but not for non-sync providers like GitHub.

Written by Cursor Bugbot for commit dac6481. This will update automatically on new commits. Configure here.

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 30, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
app Ready Ready Preview, Comment Apr 2, 2026 8:01pm
comp-framework-editor Ready Ready Preview, Comment Apr 2, 2026 8:01pm
portal Ready Ready Preview, Comment Apr 2, 2026 8:01pm

Request Review

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

@Marfuen Marfuen merged commit 6ab00a5 into main Apr 2, 2026
8 of 10 checks passed
@Marfuen Marfuen deleted the eng-114-prompt-import-after-google-workspace branch April 2, 2026 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants