Repository navigation
change: [M3-8807] - Improve Linode Create VPC user experience - #11188
bnussman-akamai merged 5 commits into
Conversation
There was a problem hiding this comment.
This component is a bit crazy. It was a shared component used on Linode Create v1 and in the Linode Config dialog.
Now that Linode Create v1 is gone, we can clean this up because Linode Create v2 does not share any of this code.
Hopefully I don't break anything here in the process 😅
| /** | ||
| * A function that is called when a VPC is successfully created | ||
| */ | ||
| onSuccess: (vpc: VPC) => void; |
There was a problem hiding this comment.
I find onSuccess to be a but more understandable than handleSelectVPC
There was a problem hiding this comment.
That's fine.. our handlers naming conventions are really all over the place 😅
|
Coverage Report: ✅ |
There was a problem hiding this comment.
The diff here is a bit hard to read but I essentially just deleted any test that tested the prop from="linodeCreate". The "from" prop doesn't need to exist anymore now that Linode Create doesn't use this component
bnussman-akamai
changed the title
abailly-akamai
left a comment
abailly-akamai
left a comment
There was a problem hiding this comment.
Nice, thanks for the clarity improvements too ✅
No regressions when testing Linode creation with VPC
| /** | ||
| * A function that is called when a VPC is successfully created | ||
| */ | ||
| onSuccess: (vpc: VPC) => void; |
There was a problem hiding this comment.
That's fine.. our handlers naming conventions are really all over the place 😅
|
|
||
| try { | ||
| const response = await createVPC(createVPCPayload); | ||
| const vpc = await createVPC(createVPCPayload); |
There was a problem hiding this comment.
✅ confirmed updated unit and cypress test pass + test failure in cypress run looks unrelated
✅ verified that subnet gets preselected if VPC only has one subnet, for both creating a new VPC and choosing an already existing VPC
✅ verified subnet doesn't get preselected if VPC has 2+ subnets
✅ vpc configuration panel
Thanks Banks!! This is some really nice cleanup + added clarity! 🎉
(...on a side note, now that I'm looking at the VPC Create code again, I kinda wanna clean that up + switch it all to react-hook-form too... gonna try to get that in before VPC part 2 starts up!! 🤞🙏 )
4 participants
Description 📝
Preview 📷
Screen.Recording.2024-10-29.at.8.40.03.PM.mov
Screen.Recording.2024-10-29.at.8.41.15.PM.mov
How to test 🧪
As an Author I have considered 🤔