Skip to content

Oauth configs #359

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 10 commits into from
Closed

Oauth configs #359

wants to merge 10 commits into from

Conversation

@sebbi08
Copy link
Contributor

@sebbi08 sebbi08 commented Jul 4, 2025

Readiness checklist

  • I ensured that the PR title is good enough.
  • I labeled the PR.
  • I self-reviewed the PR.

@sebbi08 sebbi08 changed the base branch from main to release/v7 July 4, 2025 05:40
@sebbi08 sebbi08 added the enhancement New feature or request label Jul 4, 2025
@britsta britsta mentioned this pull request Jul 11, 2025
Merged
3 tasks
@jkoenig134
Copy link
Member

jkoenig134 commented Jul 15, 2025

#364 is the correct version for this that actually documents what the config in connector v7 looks like. I close this, but I will look if there is anything that I can transfer :)

@jkoenig134 jkoenig134 closed this Jul 15, 2025
@jkoenig134 jkoenig134 deleted the oauth-configs branch July 22, 2025 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tnotheis tnotheis Awaiting requested review from tnotheis tnotheis will be requested when the pull request is marked ready for review tnotheis is a code owner

@jkoenig134 jkoenig134 Awaiting requested review from jkoenig134 jkoenig134 will be requested when the pull request is marked ready for review jkoenig134 is a code owner

@stnmtz stnmtz Awaiting requested review from stnmtz stnmtz will be requested when the pull request is marked ready for review stnmtz is a code owner

Assignees

No one assigned

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@sebbi08 @jkoenig134 @britsta