-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Update mention about dynamically generated slot names #6440
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM ✅
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MoustaphaDev Let me know if you're OK with the proposed changes below, or fix if necessary.
If it's for tomorrow's minor release, I'll take responsibility for merging it, along with the other docs!
Co-authored-by: Sarah Rainsberger <[email protected]>
I said I would add an example, but I think it's okay to keep it as-is, for now at least. |
Sounds perfect, thank you @MoustaphaDev ! NWTWWHB! |
* Update with PR #6440 * Update with PR #6386 * Up with PR #6532 * Up with PR #6436 * Upd Just for Lunaria with PR #6444 * Up with PR #6402 * Up with PR #6548 * Up with PR #6430 * Up with PR #6469 * Up with PR #6522 * Up with PR #6272 * Update src/content/docs/fr/editor-setup.mdx Co-authored-by: Thomas Bonnet <[email protected]> * Update src/content/docs/fr/reference/publish-to-npm.mdx Co-authored-by: Thomas Bonnet <[email protected]> --------- Co-authored-by: Thomas Bonnet <[email protected]> Co-authored-by: Yan Thomas <[email protected]>
Co-authored-by: Sarah Rainsberger <[email protected]>
* Update with PR #6440 * Update with PR #6386 * Up with PR #6532 * Up with PR #6436 * Upd Just for Lunaria with PR #6444 * Up with PR #6402 * Up with PR #6548 * Up with PR #6430 * Up with PR #6469 * Up with PR #6522 * Up with PR #6272 * Update src/content/docs/fr/editor-setup.mdx Co-authored-by: Thomas Bonnet <[email protected]> * Update src/content/docs/fr/reference/publish-to-npm.mdx Co-authored-by: Thomas Bonnet <[email protected]> --------- Co-authored-by: Thomas Bonnet <[email protected]> Co-authored-by: Yan Thomas <[email protected]>
This reverts commit 86ac9ba.
#6793) * Revert "Update mention about dynamically generated slot names (#6440)" This reverts commit 86ac9ba. * apply sarah's suggestions Co-authored-by: Sarah Rainsberger <[email protected]> --------- Co-authored-by: Sarah Rainsberger <[email protected]>
Description
In withastro/compiler#933, we added support for dynamically generated slots. This PR updates a reference about that feature not being supported and underlines the minimum astro version required to use it
For Astro version:
4.2.0
.See astro PR withastro/astro#9605