-
Notifications
You must be signed in to change notification settings - Fork 69
feat: Implement "Create New Secret" Modal #751
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
base: notebooks-v2
Are you sure you want to change the base?
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
86bc28d to
a44fe37
Compare
workspaces/frontend/src/app/pages/Workspaces/Form/properties/secrets/SecretsApiCreateModal.tsx
Outdated
Show resolved
Hide resolved
workspaces/frontend/src/app/pages/Workspaces/Form/properties/secrets/SecretsApiCreateModal.tsx
Outdated
Show resolved
Hide resolved
workspaces/frontend/src/app/pages/Workspaces/Form/properties/secrets/SecretsApiCreateModal.tsx
Outdated
Show resolved
Hide resolved
workspaces/frontend/src/app/pages/Workspaces/Form/properties/secrets/SecretsApiCreateModal.tsx
Outdated
Show resolved
Hide resolved
| </Alert> | ||
| )} | ||
| <Form> | ||
| <ThemeAwareFormGroupWrapper label="Secret name" isRequired fieldId="secret-name"> |
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.
We should include a dropdown here to select a Secret Type. The only type supported is "Opaque", but ideally we would want to conditionally render the input field (i.e. if Opaque, then display Value-Pair)
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.
Thank you @thaorell , yes, great point - added a "Secret Type dropdown" per your feedback. Since there's just one type supported at the moment I've disabled it for now. WDYT?
MUI has it's own styling for disabled dropdowns which is tracked in opendatahub-io/mod-arch-library#65 and will be handled as a followup.
28c0b71 to
fd09796
Compare
Signed-off-by: Charles Thao <[email protected]> Signed-off-by: Jenny <[email protected]>
Signed-off-by: Jenny <[email protected]>
Signed-off-by: Jenny <[email protected]> remove unused div and inline styles Signed-off-by: Jenny <[email protected]>
Signed-off-by: Jenny <[email protected]>
fd09796 to
8dd4a78
Compare
Closes #661
Related: #667 , #698