Skip to content

feat(ENG-11506): initiate OAuth flow with integration id#64

Merged
adefreitas merged 1 commit intoStackOneHQ:mainfrom
adefreitas:ENG-11506/send-integration-id-on-save-oauth
Nov 27, 2025
Merged

feat(ENG-11506): initiate OAuth flow with integration id#64
adefreitas merged 1 commit intoStackOneHQ:mainfrom
adefreitas:ENG-11506/send-integration-id-on-save-oauth

Conversation

@adefreitas
Copy link
Copy Markdown
Collaborator

@adefreitas adefreitas commented Nov 27, 2025

Summary by cubic

Switch OAuth initiation to use integration_id in the URL path and remove the provider_version param, so the new hub sends the integration ID during account creation. Aligns with ENG-11506 and ensures the OAuth flow routes to the correct integration.

Written for commit 1adc381. Summary will update automatically on new commits.

Copilot AI review requested due to automatic review settings November 27, 2025 09:51
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the OAuth flow to use integration_id instead of provider as the URL parameter, and removes the provider_version query parameter from the OAuth redirect URL.

  • Changed OAuth endpoint from provider-based to integration_id-based routing
  • Removed provider_version query parameter from OAuth URL construction

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

Copy link
Copy Markdown
Contributor

@joeStackOne joeStackOne left a comment

Choose a reason for hiding this comment

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

LGTM

@adefreitas adefreitas merged commit 2a817ff into StackOneHQ:main Nov 27, 2025
9 checks passed
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.

3 participants