Skip to content

feat(source-gong): migrate OAuth scope to scopes object array#75511

Closed
Serhii Lazebnyi (lazebnyi) wants to merge 1 commit intomasterfrom
slazebnyi/source-gong-scopes-array
Closed

feat(source-gong): migrate OAuth scope to scopes object array#75511
Serhii Lazebnyi (lazebnyi) wants to merge 1 commit intomasterfrom
slazebnyi/source-gong-scopes-array

Conversation

@lazebnyi
Copy link
Copy Markdown
Contributor

Summary

  • Convert scope string to scopes array format with {{scopes_param}} template variable
  • Replaces scope: api:calls:read:basic api:calls:read:extensive... with individual scope objects
  • Updates consent URL from {{scope_value | urlEncode}} to {{scopes_param}}
  • Version bump 0.6.2 → 0.6.3

Related: https://github.com/airbytehq/airbyte-internal-issues/issues/16023
Supersedes: #75195

Test plan

  • Verify consent URL includes all 5 scopes URL-encoded
  • Verify full OAuth flow: consent → callback → token exchange → access_token + refresh_token
  • Confirm scopes match production Gong OAuth configuration

Convert scope string to scopes array format with {{scopes_param}}
template variable, matching the pattern used by zendesk-chat, monday,
and other connectors.

Related: airbytehq/airbyte-internal-issues#16023
@github-actions
Copy link
Copy Markdown
Contributor

👋 Greetings, Airbyte Team Member!

Here are some helpful tips and reminders for your convenience.

💡 Show Tips and Tricks

PR Slash Commands

Airbyte Maintainers (that's you!) can execute the following slash commands on your PR:

  • 🛠️ Quick Fixes
    • /format-fix - Fixes most formatting issues.
    • /bump-version - Bumps connector versions, scraping changelog description from the PR title.
  • ❇️ AI Testing and Review (internal link: AI-SDLC Docs):
    • /ai-prove-fix - Runs prerelease readiness checks, including testing against customer connections.
    • /ai-canary-prerelease - Rolls out prerelease to 5-10 connections for canary testing.
    • /ai-review - AI-powered PR review for connector safety and quality gates.
  • 🚀 Connector Releases:
    • /publish-connectors-prerelease - Publishes pre-release connector builds (tagged as {version}-preview.{git-sha}) for all modified connectors in the PR.
    • /bump-progressive-rollout-version - Bumps connector version with an RC suffix (2.16.10-rc.1) for progressive rollouts (enableProgressiveRollout: true).
      • Example: /bump-progressive-rollout-version changelog="Add new feature for progressive rollout"
  • ☕️ JVM connectors:
    • /update-connector-cdk-version connector=<CONNECTOR_NAME> - Updates the specified connector to the latest CDK version.
      Example: /update-connector-cdk-version connector=destination-bigquery
  • 🐍 Python connectors:
    • /poe connector source-example lock - Run the Poe lock task on the source-example connector, committing the results back to the branch.
    • /poe source example lock - Alias for /poe connector source-example lock.
    • /poe source example use-cdk-branch my/branch - Pin the source-example CDK reference to the branch name specified.
    • /poe source example use-cdk-latest - Update the source-example CDK dependency to the latest available version.
  • ⚙️ Admin commands:
    • /force-merge reason="<REASON>" - Force merges the PR using admin privileges, bypassing CI checks. Requires a reason.
      Example: /force-merge reason="CI is flaky, tests pass locally"
📚 Show Repo Guidance

Helpful Resources

📝 Edit this welcome message.

@github-actions
Copy link
Copy Markdown
Contributor

source-gong Connector Test Results

5 tests   2 ✅  7s ⏱️
1 suites  3 💤
1 files    0 ❌

Results for commit db484ed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants