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-6961] - Swap VLAN and User Data order in Linode Create #9492

Merged

Conversation

hana-akamai
Copy link
Contributor

Description 📝

We got feedback from Brian Clay to display the User Data accordion after VLAN "so the customer can configure all their networking aspects in a single flow"

e.g. VPC > Firewall > VLAN > User Data instead of VPC > Firewall > User Data > VLAN

Preview 📷

image

How to test 🧪

yarn test AddonsPanel
  • Go to Linode Create
  • Select a cloud-init compatible image
  • Select a Metadata compatible region (Washington DC or Paris)

@hana-akamai hana-akamai added the VPC Relating to VPC project label Aug 4, 2023
@hana-akamai hana-akamai self-assigned this Aug 4, 2023
@bnussman-akamai bnussman-akamai added Add'tl Approval Needed Waiting on another approval! and removed Ready for Review labels Aug 4, 2023
@mjac0bs mjac0bs added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Aug 4, 2023
@hana-akamai hana-akamai merged commit 463640f into linode:develop Aug 7, 2023
@hana-akamai hana-akamai deleted the M3-6961-swap-vlan-user-data-order branch August 7, 2023 15:11
jaalah-akamai pushed a commit that referenced this pull request Aug 8, 2023
## Description 📝
We got feedback from Brian Clay to display the User Data accordion after VLAN "so the customer can configure all their networking aspects in a single flow"

e.g. `VPC > Firewall > VLAN > User Data` instead of `VPC > Firewall > User Data > VLAN`

## How to test 🧪
```
yarn test AddonsPanel
```
- Go to Linode Create
- Select a cloud-init compatible image
- Select a Metadata compatible region (Washington DC or Paris)
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