Skip to content

Conversation

@raimund-schluessler
Copy link
Contributor

Since master is 8.0.0 now, we can merge all breaking changes scheduled for the next major.

This PR removes the nameTitleFallback and makes the name prop required for NcActions*, NcAppNavigationItem and NcBreadcrumb*.

Signed-off-by: Raimund Schlüßler <[email protected]>
@raimund-schluessler raimund-schluessler added the 3. to review Waiting for reviews label May 7, 2023
@raimund-schluessler raimund-schluessler added this to the 8.0.0 milestone May 7, 2023
Copy link
Contributor

@skjnldsv skjnldsv left a comment

Choose a reason for hiding this comment

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

Could you be any more awesome Raimund? 😍
🙇

@skjnldsv skjnldsv merged commit 8d9ad47 into master May 9, 2023
@skjnldsv skjnldsv deleted the fix/noid/name-title-deprecation branch May 9, 2023 06:11
@ShGKme
Copy link
Contributor

ShGKme commented May 9, 2023

Should we start to make breaking changes list or migration guide?

@nickvergessen
Copy link
Contributor

A migration guide similar to the OCP changes sounds like a wunderful idea!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews 💥 breaking PR that requires a new major version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants