Skip to content

upcoming: [UIE-9730] - Add Beta/New Feature Chip Support for RuleSets and Prefix Lists#13164

Merged
pmakode-akamai merged 11 commits intolinode:developfrom
pmakode-akamai:UIE-9730-add-beta-new-feature-chip-support-for-ruleset-prefixlist
Dec 5, 2025
Merged

upcoming: [UIE-9730] - Add Beta/New Feature Chip Support for RuleSets and Prefix Lists#13164
pmakode-akamai merged 11 commits intolinode:developfrom
pmakode-akamai:UIE-9730-add-beta-new-feature-chip-support-for-ruleset-prefixlist

Conversation

@pmakode-akamai
Copy link
Copy Markdown
Contributor

@pmakode-akamai pmakode-akamai commented Dec 4, 2025

Description 📝

Add Beta/New Feature Chip Support for RuleSets and Prefix Lists

Changes 🔄

  • Added a utility that can be reused to display the feature chip based on the feature flag hook

Scope 🚢

  • All customers
  • Some customers (e.g. in Beta or Limited Availability)
  • No customers / Not applicable

Target release date 🗓️

N/A

Preview 📷

Before After
Screenshot 2025-12-04 at 12 33 57 PM Screenshot 2025-12-04 at 12 33 09 PM Screenshot 2025-12-04 at 12 33 44 PM Screenshot 2025-12-04 at 12 36 10 PM Screenshot 2025-12-04 at 12 37 42 PM Screenshot 2025-12-04 at 12 39 32 PM

How to test 🧪

Prerequisites

  • Enabled 'Firewall Rulesets and Prefixlists feature flag'

Verification steps

  • Verify that the chip is visible only when the feature is enabled and the relevant beta/GA flags are active
  • Verify that chips are added to all relevant places related to this feature (refer Figma design for this)
  • Ensure there is not feature chip in title in Edit mode
  • Ensure that only one feature chip is shown at a time, even when both the relevant beta and GA flags are active
Author Checklists

As an Author, to speed up the review process, I considered 🤔

👀 Doing a self review
❔ Our contribution guidelines
🤏 Splitting feature into small PRs
➕ Adding a changeset
🧪 Providing/improving test coverage
🔐 Removing all sensitive information from the code and PR description
🚩 Using a feature flag to protect the release
👣 Providing comprehensive reproduction steps
📑 Providing or updating our documentation
🕛 Scheduling a pair reviewing session
📱 Providing mobile support
♿ Providing accessibility support

  • I have read and considered all applicable items listed above.

As an Author, before moving this PR from Draft to Open, I confirmed ✅

  • All tests and CI checks are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@pmakode-akamai pmakode-akamai self-assigned this Dec 4, 2025
@pmakode-akamai pmakode-akamai added the Firewalls Related to Firewalls label Dec 4, 2025
@pmakode-akamai pmakode-akamai marked this pull request as ready for review December 4, 2025 07:49
@pmakode-akamai pmakode-akamai requested review from a team as code owners December 4, 2025 07:49
@pmakode-akamai pmakode-akamai requested review from fabrice-akamai and grevanak-akamai and removed request for a team December 4, 2025 07:49
Copy link
Copy Markdown
Contributor

@harsh-akamai harsh-akamai left a comment

Choose a reason for hiding this comment

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

Thanks @pmakode-akamai!

@harsh-akamai
Copy link
Copy Markdown
Contributor

I believe this PR is missing a changeset. Please do add it before merging

@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Dec 4, 2025
@grevanak-akamai grevanak-akamai removed the Add'tl Approval Needed Waiting on another approval! label Dec 4, 2025
isFirewallRulesetsPrefixListsGAEnabled,
}) ?? undefined;

// Do not show the Firewall RS & PL feature chip in Edit mode drawer title
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Good that we commented the condition 👍

@dwiley-akamai dwiley-akamai self-requested a review December 4, 2025 14:17
Copy link
Copy Markdown
Contributor

@dwiley-akamai dwiley-akamai left a comment

Choose a reason for hiding this comment

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

Code review ✅
Verification steps ✅

Did program leadership provide any guidance about a Limited Availability (LA) period?

@pmakode-akamai pmakode-akamai added the Approved Multiple approvals and ready to merge! label Dec 5, 2025
@pmakode-akamai pmakode-akamai merged commit fc40927 into linode:develop Dec 5, 2025
34 of 35 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager Dec 5, 2025
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! Firewalls Related to Firewalls

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants