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: release plans nav menu item and icon suggestion #8469

Merged
merged 2 commits into from
Oct 18, 2024

Conversation

daveleek
Copy link
Contributor

@daveleek daveleek commented Oct 17, 2024

Adds a Release management item to the nav menu and an empty releasemgmt page component.
Temporarily suggests the MUI Launch icon as the release mgmt link icon.

image

Copy link

vercel bot commented Oct 17, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
unleash-monorepo-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 17, 2024 1:45pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
unleash-docs ⬜️ Ignored (Inspect) Visit Preview Oct 17, 2024 1:45pm

Copy link
Contributor

github-actions bot commented Oct 17, 2024

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Manifest Files

@gardleopard
Copy link
Contributor

Is this behind a feature flag?

component: ReleaseManagement,
type: 'protected',
menu: { advanced: true, mode: ['enterprise'] },
flag: 'releasePlans',
Copy link
Contributor Author

@daveleek daveleek Oct 17, 2024

Choose a reason for hiding this comment

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

This @gardleopard makes sure it's shown only when the specified flag is 'true'

  • Another interesting side effect, setting that flag to true negates any enterprise check for the link, so would show up in both OSS and Pro, but the route will still yield a 404 in that case.

Still, it's something we might want to look at (but I feel that's beyond the scope of this PR, I'm making a bug task for it though)

Copy link
Contributor

@gardleopard gardleopard left a comment

Choose a reason for hiding this comment

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

LGTM

@daveleek daveleek merged commit d9646ed into main Oct 18, 2024
12 checks passed
@daveleek daveleek deleted the feat/release-plans-nav-item branch October 18, 2024 06:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants