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: Remove Beta status from Header and deprecate Mega Menu #1814

Merged
merged 4 commits into from
Jan 17, 2025

Conversation

VincentSmedinga
Copy link
Contributor

The Header component is no longer in Beta.

The Mega Menu component is deprecated. I’ve updated the Status Badge component to allow customizing the status label and making it red for a deprecation.

@@ -20,4 +20,5 @@ const MegaMenuRoot = forwardRef(

MegaMenuRoot.displayName = 'MegaMenu'

/** @deprecated Use child components in Header instead. */
Copy link
Contributor Author

@VincentSmedinga VincentSmedinga Jan 13, 2025

Choose a reason for hiding this comment

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

We probably want this deprecation to appear in the change log. Where should we list it, under ‘Features’?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, I think so. It's a deprecation of a feature.

@@ -20,4 +20,5 @@ const MegaMenuRoot = forwardRef(

MegaMenuRoot.displayName = 'MegaMenu'

/** @deprecated Use child components in Header instead. */
Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, I think so. It's a deprecation of a feature.

@alimpens alimpens changed the title chore: Remove Beta status from Header and deprecate Mega Menu feat: Remove Beta status from Header and deprecate Mega Menu Jan 17, 2025
@alimpens alimpens requested a review from a team as a code owner January 17, 2025 15:47
@github-actions github-actions bot temporarily deployed to demo-update-header-and-mega-menu-statuses January 17, 2025 15:47 Destroyed
@alimpens alimpens enabled auto-merge (squash) January 17, 2025 15:47
@alimpens alimpens merged commit c91c9d0 into develop Jan 17, 2025
6 checks passed
@alimpens alimpens deleted the chore/update-header-and-mega-menu-statuses branch January 17, 2025 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants