Skip to content

fix(seer): RCA Onboarding step broken due to provider#107680

Merged
billyvg merged 1 commit intomasterfrom
billy/fix-root-cause-analysis-provider-invalid
Feb 5, 2026
Merged

fix(seer): RCA Onboarding step broken due to provider#107680
billyvg merged 1 commit intomasterfrom
billy/fix-root-cause-analysis-provider-invalid

Conversation

@billyvg
Copy link
Member

@billyvg billyvg commented Feb 5, 2026

I dont know yet if something changed but provider name was being sent e.g. (GitHub), but its all lowercase in the db

repo.id will be something like integrations:github which is what we want vs name which is GitHub (backend checks both using the provider property (e.g. matches ${provider} and integrations:${provider}

I dont know yet if something changed but provider name was being sent e.g. (GitHub), but its all lowercase in the db

repo.id will be something like `integrations:github` which is what we want vs name which is `GitHub` (backend checks both using the `provider` property (e.g. matches `${provider}` and `integrations:${provider}`
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Feb 5, 2026
@billyvg billyvg marked this pull request as ready for review February 5, 2026 16:52
@billyvg billyvg requested a review from a team as a code owner February 5, 2026 16:52
@ryan953
Copy link
Member

ryan953 commented Feb 5, 2026

any other places using the wrong thing? i believe i put in some strings like "github" in a 1-2 settings related places

@billyvg
Copy link
Member Author

billyvg commented Feb 5, 2026

I see two places use it but they force lowercase on it

@billyvg billyvg merged commit 44d3822 into master Feb 5, 2026
59 checks passed
@billyvg billyvg deleted the billy/fix-root-cause-analysis-provider-invalid branch February 5, 2026 17:00
billyvg added a commit that referenced this pull request Feb 5, 2026
jaydgoss pushed a commit that referenced this pull request Feb 12, 2026
I dont know yet if something changed but provider name was being sent
e.g. (GitHub), but its all lowercase in the db

repo.id will be something like `integrations:github` which is what we
want vs name which is `GitHub` (backend checks both using the `provider`
property (e.g. matches `${provider}` and `integrations:${provider}`
jaydgoss pushed a commit that referenced this pull request Feb 12, 2026
dcramer pushed a commit that referenced this pull request Feb 17, 2026
I dont know yet if something changed but provider name was being sent
e.g. (GitHub), but its all lowercase in the db

repo.id will be something like `integrations:github` which is what we
want vs name which is `GitHub` (backend checks both using the `provider`
property (e.g. matches `${provider}` and `integrations:${provider}`
dcramer pushed a commit that referenced this pull request Feb 17, 2026
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