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

[next] feat(NcCounterBubble): add count prop for humanized count display #5990

Merged
merged 1 commit into from
Aug 20, 2024

Conversation

backportbot[bot]
Copy link

@backportbot backportbot bot commented Aug 20, 2024

Backport of PR #5863

@backportbot backportbot bot added enhancement New feature or request 3. to review Waiting for reviews deprecation Related to the deprecation of anything available on the public API labels Aug 20, 2024
@ShGKme ShGKme marked this pull request as draft August 20, 2024 10:42
@Antreesy
Copy link
Contributor

Don't forget to include #5985 =)

@ShGKme
Copy link
Contributor

ShGKme commented Aug 20, 2024

Don't forget to include #5985 =)

I won't include, it's a different PR ._.

@ShGKme ShGKme added breaking PR that requires a new major version and removed deprecation Related to the deprecation of anything available on the public API labels Aug 20, 2024
@ShGKme ShGKme marked this pull request as ready for review August 20, 2024 17:43
*/
raw: {
type: Boolean,
required: false,
Copy link
Contributor

Choose a reason for hiding this comment

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

not needed

Copy link
Contributor

Choose a reason for hiding this comment

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

Why?

Copy link
Contributor

Choose a reason for hiding this comment

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

All props are optional by default, unless required: true is specified.

@susnux
Copy link
Contributor

susnux commented Aug 20, 2024

I won't include, it's a different PR ._.

then do not forget to backport ;)

@ShGKme
Copy link
Contributor

ShGKme commented Aug 20, 2024

I won't include, it's a different PR ._.

then do not forget to backport ;)

I won't. One by one. That PR is on made on top of this

@ShGKme
Copy link
Contributor

ShGKme commented Aug 20, 2024

Hmm, It also make sense to not mix new huge change (removing API) to a backport

@ShGKme
Copy link
Contributor

ShGKme commented Aug 20, 2024

Kept backport clean with the backport only

@ShGKme ShGKme removed the breaking PR that requires a new major version label Aug 20, 2024
@ShGKme ShGKme merged commit 11a63c0 into next Aug 20, 2024
20 checks passed
@ShGKme ShGKme deleted the backport/5863/next branch August 20, 2024 20:27
@ShGKme ShGKme changed the title [next] feat(NcCounterBubble): add count and limit props instead of slot [next] feat(NcCounterBubble): add count prop for humanized count display Aug 20, 2024
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 enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants