-
Notifications
You must be signed in to change notification settings - Fork 401
upcoming: [UIE-8515] - DBaaS: Advanced Configuration - Drawer with existing configs #11812
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
Changes from 7 commits
fb80157
b18cae6
0586548
50f0736
1ad772d
cd428e4
2f2b1a6
4b9edf3
4eae4c9
1ff25f8
d768e29
6c8a307
a35b87d
9283e2e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| --- | ||
| "@linode/api-v4": Added | ||
| --- | ||
|
|
||
| DBaaS Advanced Configurations: added `getDatabaseEngineConfig` request to fetch all advanced configurations and updated types for advanced configs ([#11812](https://github.com/linode/manager/pull/11812)) |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| --- | ||
| "@linode/manager": Upcoming Features | ||
| --- | ||
|
|
||
| DBaaS Advanced Configurations: added UI for existing engine options in the drawer ([#11812](https://github.com/linode/manager/pull/11812)) |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -5,8 +5,29 @@ import { StyledValueGrid } from '../DatabaseSummary/DatabaseSummaryClusterConfig | |
| export const StyledConfigValue = styled(StyledValueGrid, { | ||
| label: 'StyledValueGrid', | ||
| })(({ theme }) => ({ | ||
| padding: `${theme.spacing(0.5)} | ||
| ${theme.spacing(1.9)} | ||
| ${theme.spacing(0.5)} | ||
| ${theme.spacing(0.8)}`, | ||
| padding: `${theme.tokens.spacing.S4} ${theme.tokens.spacing.S6}`, | ||
| })); | ||
|
|
||
| export const GroupHeader = styled('div')(({ theme }) => ({ | ||
| background: theme.tokens.background.Neutral, | ||
| color: | ||
| theme.palette.mode === 'dark' | ||
| ? theme.tokens.color.Neutrals[5] | ||
| : theme.tokens.color.Neutrals[100], | ||
| font: theme.tokens.typography.Label.Bold.Xs, | ||
| padding: '8px 12px', | ||
| position: 'sticky', | ||
| textTransform: 'uppercase', | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. same
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I will update this in the next PR, as these styles are for Autocomplete, which will be implemented in the next PR. |
||
| top: 0, | ||
| zIndex: 1, | ||
| })); | ||
| export const GroupItems = styled('ul')(({ theme }) => ({ | ||
| '& li': { | ||
| color: | ||
| theme.palette.mode === 'dark' | ||
| ? theme.tokens.color.Neutrals[5] | ||
| : theme.tokens.color.Neutrals[100], | ||
| font: theme.tokens.typography.Label.Regular.Xs, | ||
| }, | ||
|
Comment on lines
+11
to
+31
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Wondering where these (
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thank you for the review! This will be used for 'Add configuration' Autocomplete. I had to split the second PR into two parts because it was too large, so the rest will be in the next PR. |
||
| padding: 0, | ||
| })); | ||
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.
DBaaS is a good candidate to move to the new mock service. Is there a ticket/scope for it?