-
Notifications
You must be signed in to change notification settings - Fork 13.4k
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
fix(editor): Load only personal credentials when setting up a template #12826
fix(editor): Load only personal credentials when setting up a template #12826
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅ 📢 Thoughts on this report? Let us know! |
|
|
1 similar comment
|
|
✅ All Cypress E2E specs passed |
Got released with |
Summary
Templates are created in Personal by default, so we only need to show personal credentials in the setup dialog.
NOTE: For now, the
CredentialPicker
component is only used in the template setup so this quick fix makes sense. Once we decide to reuse it it will make sense to pass project/owner as a prop so we can do custom filtering.Related Linear tickets, Github issues, and Community forum posts
Fixes ADO-2984
Review / Merge checklist
release/backport
(if the PR is an urgent fix that needs to be backported)