Hana Xu

Results 9 comments of Hana Xu

It looks like VPC isn't supported 🤔 | This branch | Prod | | ------- | ------- | | ![image](https://github.com/linode/manager/assets/115299789/bce71eb6-8d9d-4b2f-a9d7-15ad47164dcc) | ![image](https://github.com/linode/manager/assets/115299789/1b7bd10a-0a48-4aab-b490-f69f03bb1880) | | ![image](https://github.com/linode/manager/assets/115299789/f516cc5f-19da-4d01-a7f8-d545eae5a496) | ![image](https://github.com/linode/manager/assets/115299789/887b9bdd-7095-4173-b083-f37647a2d2ac) |

@bnussman looks like something is off with the metadata encoding https://www.base64decode.org/ | This branch | Prod | | ------- | ------- | | ![image](https://github.com/linode/manager/assets/115299789/c80e8261-1555-4c9d-8450-c0cdccc76a1b) | ![image](https://github.com/linode/manager/assets/115299789/0f97fc72-ee40-455f-933e-089b014976e1) |

@bnussman-akamai Good point, just realized we don't use the `TableRowLoading` component in `LinodesLanding`. I'll create a separate ticket to address that since it's going to require some code maneuvering Edit:...

Should the onBlur / clicking `Create` in the VLAN dropdown send a request to create the VLAN without having to wait until the user clicks `Create Linode`?

I think we should keep some kind of red tooltip error icon instead of just replacing across the board with a warning icon as there are semantic differences For example,...

@tzmiivsk-akamai I think the only case where we use the error Tooltip icon was in the Add Payment Drawer (seen in the description screenshot). Since it's a one-off we can...

@jdamore-linode this is still a WIP so things are probably going to change, I'll let you know when this is ready for review!

Thanks @jdamore-linode! Is the `cy.get('main').within(() => {...` syntax preferred over using `data-testid`?