manager
manager copied to clipboard
fix: [M3-8492] - Create Linode Firewall warning not appearing
Description ๐
Fixes a bug causing the Firewall warning to not appear after the user assigns and then clears the firewall select during the create flow.
Changes ๐
- Update the
firewall_idinterface inCreateLinodePayloadto not accept null values (i.e.,number | undefined) - Refactor
SelectFirewallPaneland associated types in Create v1 to useundefinedinstead of-1when the firewall field is cleared - Remove a redundant
useEffect
Target release date ๐๏ธ
9/3
Preview ๐ท
| Before | After |
|---|---|
| Create v2 | |
| Create v1 | |
How to test ๐งช
Prerequisites
Enable Secure VM notices by adding secure_vm_notices: 'always' in the preference editor.
Reproduction steps
- Select and then clear the firewall field in the Linode Create flows (v1 and v2)
- Click the "Create Linode" button
- Observe that the Firewall authorization warning banner does not appear
- In Create v1, the form is submitted and the Linode is created
- In Create v2, the Create Linode button becomes disabled
Verification steps
- Follow the reproduction steps
- Verify that the Firewall authorization banner always appears after clicking the Create Linode button when no firewall is assigned
Coverage Report: โ
Base Coverage: 82.73%
Current Coverage: 82.74%