Skip to content

refactor(NcChip): use defineSlots for proper slot definition #6792

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

susnux
Copy link
Contributor

@susnux susnux commented Apr 16, 2025

☑️ Resolves

This resolves 2 errors reported by Typescript when building the package (missing "private" type exports during the declaration creation).

🏁 Checklist

  • ⛑️ Tests are included or are not applicable
  • 📘 Component documentation has been extended, updated or is not applicable
  • 2️⃣ Backport to stable8 for maintained Vue 2 version or not applicable

@susnux susnux requested review from st3iny, skjnldsv and ShGKme April 16, 2025 12:46
@susnux susnux added 3. to review Waiting for reviews refactor ♻️ Pull request that is neither a fix nor a feature labels Apr 16, 2025
@susnux susnux added this to the 9.0.0 milestone Apr 16, 2025
@susnux susnux force-pushed the chore/nc-chip-ts branch 2 times, most recently from a168318 to 729dfc8 Compare April 16, 2025 12:49
@susnux susnux force-pushed the chore/nc-chip-ts branch from 729dfc8 to 6c2b368 Compare April 16, 2025 18:35
@susnux susnux requested a review from ShGKme April 16, 2025 18:35
This resolves 2 errors reported by Typescript when building the package
(missing "private" type exports during the declaration creation).

Signed-off-by: Ferdinand Thiessen <[email protected]>
@susnux susnux force-pushed the chore/nc-chip-ts branch from 6c2b368 to e6c02b8 Compare April 17, 2025 06:00
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 refactor ♻️ Pull request that is neither a fix nor a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants