-
Notifications
You must be signed in to change notification settings - Fork 33
Llb/password only credential #3067
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
d006336 to
7dd031f
Compare
7dd031f to
6aea6df
Compare
ZedLi
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work!
DhariniJeeva
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks great!
priya-patel04
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work! 🎉
6aea6df to
0afe551
Compare
hashicc
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀 Nice test refactors, thanks for updating constants, lots of great work here
* feat: 🎸 add new password type to cred models + mirage * test: 💍 fix * refactor: 💡 update target mirage trait * chore: 🤖 add missing attr type
* refactor: 💡 credential type radio field to be select field * refactor: 💡 radio component to be select component * test: 💍 fix e2e test page * test: 💍 add correct selector to e2e test * test: 💍 fix
* feat: 🎸 add password credential form * test: 💍 add a11y ignores * test: 💍 fix e2e tests * test: 💍 add password cred type to targets brokering tests * chore: 🤖 remove comment
* refactor: 💡 update credential-library form for password type * style: 💄 fix display issue * test: 💍 add test cases
0afe551 to
accf9fa
Compare
Description
Add
passwordcredential type to credentials andpasswordcredential type to vault generic secrets credential library.Checklist
a11y-testslabel to run a11y audit tests if neededPCI review checklist
Examples of changes to security controls include using new access control methods, adding or removing logging pipelines, etc.