Skip to content
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.

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

Split oauthconfig per document #15782

Merged
merged 12 commits into from
Mar 21, 2025
Merged

Conversation

adrinr
Copy link
Collaborator

@adrinr adrinr commented Mar 20, 2025

Description

As discussed on the 1:1, splitting the oauthconfig to each own document instead of having an aggregate

@adrinr adrinr requested a review from a team as a code owner March 20, 2025 16:53
@adrinr adrinr requested review from samwho and removed request for a team March 20, 2025 16:53
Copy link

linear bot commented Mar 20, 2025

Copy link

qa-wolf bot commented Mar 20, 2025

QA Wolf here! As you write new code it's important that your test coverage is keeping up.
Click here to request test coverage for this PR!

@adrinr adrinr requested review from mike12345567 and removed request for samwho March 20, 2025 16:53
@github-actions github-actions bot added firestorm Data/Infra/Revenue Team size/m labels Mar 20, 2025
Copy link
Collaborator

@mike12345567 mike12345567 left a comment

Choose a reason for hiding this comment

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

LGTM!

@adrinr adrinr enabled auto-merge March 21, 2025 08:08
@adrinr adrinr merged commit 35335a0 into master Mar 21, 2025
22 checks passed
@adrinr adrinr deleted the BUDI-9127/split-oauthconfig-per-document branch March 21, 2025 08:18
@github-actions github-actions bot locked and limited conversation to collaborators Mar 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
firestorm Data/Infra/Revenue Team size/m
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants