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

feat: [UIE-8002] - DBaaS 2.0 Landing Page #10823

Conversation

mpolotsk-akamai
Copy link
Contributor

@mpolotsk-akamai mpolotsk-akamai commented Aug 23, 2024

Description 📝

DBaaS 2.0 Landing Page and Empty Landing Page

Changes 🔄

  • Added Tabs to the Landing Page: Two tabs for Beta customers, labeled "Legacy Databases" and "New Databases."
  • Added Logo(including dark theme): Displayed on the Empty Landing Page, in the "New Databases" tab, and in the "New Databases" single table view.
  • Single Table Landing Page: Added for Beta customers without "Legacy Databases" but with "New Databases," and for new customers with only "New Databases."
  • New "Plan" Column: Added to the "New Databases" table.
  • Added Info Banner: Included on the landing page.

Target release date 🗓️

9/9/24

Preview 📷

Include a screenshot or screen recording of the change

💡 Use <video src="" /> tag when including recordings in table.

Before After
Screenshot 2024-08-23 at 2 21 42 PM Screenshot 2024-08-23 at 2 20 46 PM
Screenshot 2024-08-23 at 2 17 50 PM Screenshot 2024-08-23 at 2 18 05 PM
Screenshot 2024-08-23 at 5 40 13 PM
Screenshot 2024-08-23 at 2 22 56 PM

How to test 🧪

Note:
To test all beta features, beta must be enabled, and the user must have the "Managed Databases Beta" account capability.

Prerequisites

(How to setup test environment)
Enable DatabaseV2Beta flag

Verification steps

(How to verify changes)

  1. Navigate to the Databases section.
  2. For new customers (with no legacy or new databases), the Landing Page will display an Empty Landing Page with a Logo.
  3. For customers with Legacy Databases, the Landing Page will show tabs for "Legacy Databases" and "New Databases," along with an info banner.
  4. When "New Databases" tab is selected, the page will display a table with an additional "Plan" column and a logo below the table. The "Configuration" column will be renamed to "Nodes." The table will be empty if no new databases are present.
  5. For customers without Legacy Databases but with New Databases, the Landing Page will show a single table with a "Plan" column and the "Configuration" column renamed to "Nodes," an info banner, and a Logo below the table.
  6. If the DatabaseV2Beta flag is disabled, the Landing Page will display a single table with no changes.

As an Author I have considered 🤔

Check all that apply

  • 👀 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

@mpolotsk-akamai mpolotsk-akamai requested a review from a team as a code owner August 23, 2024 15:43
@mpolotsk-akamai mpolotsk-akamai requested review from mjac0bs and harsh-akamai and removed request for a team August 23, 2024 15:43
@mjac0bs
Copy link
Contributor

mjac0bs commented Aug 23, 2024

@mpolotsk-akamai I just merged in #10786 so we'll want to rebase this PR on develop.

@mjac0bs mjac0bs added the DBaaS Relates to Database as a Service label Aug 23, 2024
@mpolotsk-akamai mpolotsk-akamai force-pushed the feature/UIE-8002-New-Customer-Flow-Landing-Page branch 2 times, most recently from a837c8e to ea91214 Compare August 26, 2024 11:45
@mpolotsk-akamai
Copy link
Contributor Author

@mpolotsk-akamai I just merged in #10786 so we'll want to rebase this PR on develop.

@mjac0bs , thanks, I've rebased the PR

Copy link

github-actions bot commented Aug 26, 2024

Coverage Report:
Base Coverage: 86.63%
Current Coverage: 86.63%

@mpolotsk-akamai mpolotsk-akamai force-pushed the feature/UIE-8002-New-Customer-Flow-Landing-Page branch 3 times, most recently from 8fee671 to a77a2ec Compare August 28, 2024 16:42
@mpolotsk-akamai mpolotsk-akamai force-pushed the feature/UIE-8002-New-Customer-Flow-Landing-Page branch from a77a2ec to f615f52 Compare August 28, 2024 17:50
@mpolotsk-akamai mpolotsk-akamai force-pushed the feature/UIE-8002-New-Customer-Flow-Landing-Page branch from 4952395 to 6962afe Compare August 29, 2024 16:17
Copy link
Contributor

@mjac0bs mjac0bs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for addressing feedback, @mpolotsk-akamai - I just left a couple of minor notes on my final pass through.

CI was struggling to run the full e2e suite but I'll make sure DBaaS is all passing before this gets merged.

@mjac0bs mjac0bs added the Add'tl Approval Needed Waiting on another approval! label Aug 30, 2024
@mjac0bs
Copy link
Contributor

mjac0bs commented Aug 30, 2024

@mpolotsk-akamai Just noticed that typecheck is was failing - using the optional chaining operator to access databases?.data should fix that where there are errors.

Copy link
Member

@bnussman-akamai bnussman-akamai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is shaping up nicely! Will continue to review, but left a few more small comments in the meantime

@mpolotsk-akamai mpolotsk-akamai force-pushed the feature/UIE-8002-New-Customer-Flow-Landing-Page branch from 6962afe to 910e0df Compare September 5, 2024 08:26
@mpolotsk-akamai mpolotsk-akamai force-pushed the feature/UIE-8002-New-Customer-Flow-Landing-Page branch from 910e0df to 87ba377 Compare September 5, 2024 10:28
Copy link
Member

@bnussman-akamai bnussman-akamai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm seeing extra spacing here compared to production that we should fix

Screenshot 2024-09-06 at 10 01 34 AM

@mpolotsk-akamai mpolotsk-akamai force-pushed the feature/UIE-8002-New-Customer-Flow-Landing-Page branch from 87ba377 to c703764 Compare September 9, 2024 12:04
@bnussman-akamai bnussman-akamai added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Sep 9, 2024
@mpolotsk-akamai mpolotsk-akamai force-pushed the feature/UIE-8002-New-Customer-Flow-Landing-Page branch from c703764 to bac6623 Compare September 9, 2024 13:59
@mpolotsk-akamai mpolotsk-akamai force-pushed the feature/UIE-8002-New-Customer-Flow-Landing-Page branch from bac6623 to 92369fb Compare September 11, 2024 11:44
Copy link
Contributor

@jdamore-linode jdamore-linode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just want to call out the failing Cypress tests in create-database.spec.ts so we don't accidentally merge this before they're fixed.

It looks like the tests are failing because no regions are appearing in the region select drop-down -- that's probably caused by something not being mocked

@mpolotsk-akamai mpolotsk-akamai force-pushed the feature/UIE-8002-New-Customer-Flow-Landing-Page branch from ee35a62 to f4683a8 Compare September 12, 2024 14:15
@mpolotsk-akamai
Copy link
Contributor Author

Just want to call out the failing Cypress tests in create-database.spec.ts so we don't accidentally merge this before they're fixed.

It looks like the tests are failing because no regions are appearing in the region select drop-down -- that's probably caused by something not being mocked

@jdamore-linode, I've fixed the issue.

@jdamore-linode
Copy link
Contributor

Nice! Thanks @mpolotsk-akamai!

Copy link
Contributor

@jdamore-linode jdamore-linode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks again @mpolotsk-akamai!

@mpolotsk-akamai mpolotsk-akamai force-pushed the feature/UIE-8002-New-Customer-Flow-Landing-Page branch from f4683a8 to c3460e7 Compare September 13, 2024 16:27
Copy link
Contributor

@mjac0bs mjac0bs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @mpolotsk-akamai and @corya-akamai - looking good now. I'll merge this as soon as CI finishes.

@mjac0bs mjac0bs merged commit f0d9765 into linode:develop Sep 13, 2024
19 checks passed
nikhagra-akamai pushed a commit to nikhagra-akamai/manager that referenced this pull request Sep 23, 2024
* feat: [UIE-8054] - DBaaS enhancements 1

* UIE-8002 feat(DBaaS): New/Existing Customer Landing Page

* UIE-8002 feat(DBaaS): Landing Page with feature flag

* Added changeset: Add Landing Page for V2

* Added changeset: Add Landing Page and update Empty-State Landing page for DBaaS V2

* feat: [UIE-8002] - Review fix: refactoring and variable renaming

* feat: [UIE-8002] - Review fix: change capability for region select

---------

Co-authored-by: Conal Ryan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved Multiple approvals and ready to merge! DBaaS Relates to Database as a Service
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants