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: [M3-6736] - VPC detail summary #9549

Merged
merged 16 commits into from
Aug 21, 2023

Conversation

hana-akamai
Copy link
Contributor

@hana-akamai hana-akamai commented Aug 15, 2023

Description 📝

Add VPC Detail Summary section with Subnets placeholder

Preview 📷

image

image

How to test 🧪

yarn test VPCDetail
yarn test packages/manager/src/features/VPC/utils.test.ts
  • Create a VPC, then go to it's details page and confirm the summary
  • If there is no description for the VPC, the description field should not display
  • If there is a description, you should see the field
  • Try editing/deleting the VPC

@hana-akamai hana-akamai added the VPC Relating to VPC project label Aug 15, 2023
@hana-akamai hana-akamai self-assigned this Aug 15, 2023
Copy link
Contributor

@dwiley-akamai dwiley-akamai left a comment

Choose a reason for hiding this comment

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

Looks great! Left a couple of small comments but going ahead and approving 🚀

@bnussman-akamai bnussman-akamai added Add'tl Approval Needed Waiting on another approval! and removed Ready for Review labels Aug 18, 2023
@jdamore-linode
Copy link
Contributor

Screenshot 2023-08-18 at 6 41 23 PM

In the event that a user updates a VPC with a shorter description, is it expected for the layout to shift and for the "Read More" link to remain visible?

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.

Nice work @hana-linode ! Added an integration test for the details page, and approving pending the question I posted above!

@bnussman-akamai bnussman-akamai added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Aug 18, 2023
@hana-akamai hana-akamai merged commit 62a266b into linode:develop Aug 21, 2023
corya-akamai pushed a commit to corya-akamai/manager that referenced this pull request Sep 6, 2023
## Description 📝
Add VPC Detail Summary section with Subnets placeholder

## How to test 🧪
```
yarn test VPCDetail
```
```
yarn test packages/manager/src/features/VPC/utils.test.ts
```
- Create a VPC, then go to it's details page and confirm the summary
- If there is no description for the VPC, the description field should not display
- If there is a description, you should see the field
- Try editing/deleting the VPC

---------

Co-authored-by: Joe D'Amore <jdamore@linode.com>
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! VPC Relating to VPC project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants