Skip to content
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

upcoming: [UIE-8515] - DBaaS: Advanced Configuration - Drawer with existing configs #11812

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

mpolotsk-akamai
Copy link
Contributor

@mpolotsk-akamai mpolotsk-akamai commented Mar 10, 2025

Description 📝

Advanced Configuration - Drawer with existing configurations

Changes 🔄

List any change(s) relevant to the reviewer.

  • Implemented UI for the existing engine options in the drawer
  • Handled various types of configuration fields: text input, number input, select box, and checkbox
  • Added mock data for the GET request to retrieve all configurations
  • Managed form state with initial values

Target release date 🗓️

Approximate 04/02/25

Preview 📷

Before(Figma template) After
Screenshot 2025-03-10 at 1 43 16 PM Screenshot 2025-03-10 at 1 51 11 PM

How to test 🧪

Prerequisites

(How to setup test environment)

  • Database Advanced Config feature flag should be enabled

Verification steps

(How to verify changes)

  • select a database cluster
  • navigate to the Advanced Configuration Tab
  • use mock data to see table with existing configurations
  • click 'Configure' button
  • drawer with all existing configurations is shown, button 'Save and Restart Service' is disabled
  • change the configuration
  • button 'Save and Restart Service' is enabled
Author Checklists

As an Author, to speed up the review process, I considered 🤔

👀 Doing a self review
❔ Our contribution guidelines
🤏 Splitting feature into small PRs
➕ Adding a changeset
🧪 Providing/improving test coverage
🔐 Removing all sensitive information from the code and PR description
🚩 Using a feature flag to protect the release
👣 Providing comprehensive reproduction steps
📑 Providing or updating our documentation
🕛 Scheduling a pair reviewing session
📱 Providing mobile support
♿ Providing accessibility support


  • I have read and considered all applicable items listed above.

As an Author, before moving this PR from Draft to Open, I confirmed ✅

  • All unit tests are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@mpolotsk-akamai mpolotsk-akamai requested a review from a team as a code owner March 10, 2025 12:48
@mpolotsk-akamai mpolotsk-akamai requested review from carrillo-erik and hasyed-akamai and removed request for a team March 10, 2025 12:48
@mpolotsk-akamai mpolotsk-akamai self-assigned this Mar 10, 2025
@mpolotsk-akamai mpolotsk-akamai added the DBaaS Relates to Database as a Service label Mar 10, 2025
@mpolotsk-akamai mpolotsk-akamai changed the title feat: [UIE-8515] - DBaaS: Advanced Configuration - Drawer with existing configs upcoming: [UIE-8515] - DBaaS: Advanced Configuration - Drawer with existing configs Mar 10, 2025
Copy link

github-actions bot commented Mar 10, 2025

Coverage Report:
Base Coverage: 79.83%
Current Coverage: 79.83%

@cpathipa cpathipa requested review from cpathipa and removed request for carrillo-erik March 10, 2025 14:13
innodb_print_all_deadlocks: true,
service_log: false,
Copy link
Contributor

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?

@mpolotsk-akamai mpolotsk-akamai force-pushed the UIE-8515-drawer-existing-configs branch from b864aab to 50f0736 Compare March 12, 2025 12:47
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🎉 538 passing tests on test run #5 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
0 Failing538 Passing3 Skipped120m 33s

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DBaaS Relates to Database as a Service
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants