Skip to content

feat: [M3-6747] - Add filter subnets field#9647

Merged
hana-akamai merged 5 commits intolinode:developfrom
hana-akamai:M3-6747-filter-subnets-field
Sep 8, 2023
Merged

feat: [M3-6747] - Add filter subnets field#9647
hana-akamai merged 5 commits intolinode:developfrom
hana-akamai:M3-6747-filter-subnets-field

Conversation

@hana-akamai
Copy link
Contributor

@hana-akamai hana-akamai commented Sep 7, 2023

Description 📝

  • Add the ability to filter the subnets table by label or id in the VPC details page
  • Display the number of subnets in parenthesis

Preview 📷

Screen.Recording.2023-09-07.at.4.28.47.PM.mov

How to test 🧪

yarn test VPCSubnetsTable

Create some subnets in a VPC and try to filter by label/id. Ensure that table column sorting still works.

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

@abailly-akamai abailly-akamai left a comment

Choose a reason for hiding this comment

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

Works as expected 👍

We eventually may want to abstract < DebouncedSearchTextField /> and make a new component or similar since it's reused in all new designs including AGLB. Would improve reusability and make sure we are consistent with styling and debounce times

Copy link
Contributor

@coliu-akamai coliu-akamai left a comment

Choose a reason for hiding this comment

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

🎉

@coliu-akamai coliu-akamai added the Approved Multiple approvals and ready to merge! label Sep 8, 2023
@hana-akamai hana-akamai merged commit ce8a3c0 into linode:develop Sep 8, 2023
@hana-akamai hana-akamai deleted the M3-6747-filter-subnets-field branch September 8, 2023 17:45
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.

3 participants