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

Prevent passing slot names as props #8930

Merged
merged 1 commit into from
Oct 27, 2023
Merged

Prevent passing slot names as props #8930

merged 1 commit into from
Oct 27, 2023

Conversation

matthewp
Copy link
Contributor

@matthewp matthewp commented Oct 26, 2023

Changes

Testing

  • Test case added

Docs

N/A, bug fix

@changeset-bot
Copy link

changeset-bot bot commented Oct 26, 2023

🦋 Changeset detected

Latest commit: 37c73db

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added pkg: vue Related to Vue (scope) pkg: integration Related to any renderer integration (scope) pkg: astro Related to the core `astro` package (scope) pr: docs A PR that includes documentation for review labels Oct 26, 2023
Copy link
Member

@natemoo-re natemoo-re left a comment

Choose a reason for hiding this comment

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

Works for me!

Copy link
Member

@sarah11918 sarah11918 left a comment

Choose a reason for hiding this comment

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

Just a suggestion to tie the changeset description back to the behaviour reported in the original issue!

.changeset/swift-suits-drum.md Outdated Show resolved Hide resolved
@matthewp
Copy link
Contributor Author

This actually might be wrong, putting back into draft.

@matthewp matthewp marked this pull request as draft October 26, 2023 20:23
@github-actions github-actions bot removed the pkg: astro Related to the core `astro` package (scope) label Oct 26, 2023
Co-authored-by: Sarah Rainsberger <5098874+sarah11918@users.noreply.github.com>
Co-authored-by: Nate Moore <7118177+natemoo-re@users.noreply.github.com>
@matthewp matthewp marked this pull request as ready for review October 26, 2023 20:55
@matthewp
Copy link
Contributor Author

I had to go with a Vue-specific solution because we don't know what slots are going to be used inside of the component providing the slots.

@matthewp matthewp merged commit c77f55d into main Oct 27, 2023
13 checks passed
@matthewp matthewp deleted the vue-slot-rendered branch October 27, 2023 12:11
@astrobot-houston astrobot-houston mentioned this pull request Oct 27, 2023
natemoo-re added a commit that referenced this pull request Nov 22, 2023
Co-authored-by: Sarah Rainsberger <5098874+sarah11918@users.noreply.github.com>
Co-authored-by: Nate Moore <7118177+natemoo-re@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg: integration Related to any renderer integration (scope) pkg: vue Related to Vue (scope) pr: docs A PR that includes documentation for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Slot name is being rendered as an attribute on Vue component
3 participants