Skip to content

feat(pipeline): Add shared OAuth login step and redirect popup hook#111577

Merged
evanpurkhiser merged 1 commit intomasterfrom
evanpurkhiser/feat-pipeline-add-shared-oauth-login-step-and-redirect-popup-hook
Mar 26, 2026
Merged

feat(pipeline): Add shared OAuth login step and redirect popup hook#111577
evanpurkhiser merged 1 commit intomasterfrom
evanpurkhiser/feat-pipeline-add-shared-oauth-login-step-and-redirect-popup-hook

Conversation

@evanpurkhiser
Copy link
Member

Extract reusable frontend components for the OAuth popup flow so
integration pipelines can share the same authorization UX. The
OAuthLoginStep handles opening the popup, listening for postMessage
callbacks from the trampoline, and forwarding the code/state params.

Refs VDY-35

@evanpurkhiser evanpurkhiser requested a review from a team March 25, 2026 21:59
@linear-code
Copy link

linear-code bot commented Mar 25, 2026

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 25, 2026
@evanpurkhiser evanpurkhiser force-pushed the evanpurkhiser/feat-pipeline-add-shared-oauth-login-step-and-redirect-popup-hook branch from bfda548 to e1ba088 Compare March 25, 2026 22:46
@evanpurkhiser evanpurkhiser requested a review from a team as a code owner March 25, 2026 22:46
@evanpurkhiser evanpurkhiser force-pushed the evanpurkhiser/feat-pipeline-add-shared-oauth-login-step-and-redirect-popup-hook branch from e1ba088 to ebabf43 Compare March 26, 2026 14:42
Copy link
Contributor

@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 2 potential issues.

Fix All in Cursor

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

Extract reusable frontend components for the OAuth popup flow so
integration pipelines can share the same authorization UX. The
OAuthLoginStep handles opening the popup, listening for postMessage
callbacks from the trampoline, and forwarding the code/state params.

Refs VDY-35
@evanpurkhiser evanpurkhiser force-pushed the evanpurkhiser/feat-pipeline-add-shared-oauth-login-step-and-redirect-popup-hook branch from ebabf43 to e78ba50 Compare March 26, 2026 15:25
@evanpurkhiser evanpurkhiser enabled auto-merge (squash) March 26, 2026 15:27
@evanpurkhiser evanpurkhiser merged commit cbac137 into master Mar 26, 2026
70 checks passed
@evanpurkhiser evanpurkhiser deleted the evanpurkhiser/feat-pipeline-add-shared-oauth-login-step-and-redirect-popup-hook branch March 26, 2026 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants