docs
docs copied to clipboard
withastro
Add a note for using `id` instead of `slug` when using type: 'data'
Description
The slug property only exists when type is set to content. If type is set to data, using the example code will result in an error indicating that slug must be a string.
While the "Custom Slugs" and "Filtering Collection Queries" sections of this document mention that slug exists only in content, a reminder here may help beginners who overlook this point avoid trouble.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Updated (UTC) |
|---|---|---|---|
| docs | ✅ Ready (Inspect) | Visit Preview | Apr 9, 2024 7:13pm |
![vercel[bot] avatar](https://avatars.githubusercontent.com/in/8329?v=4 "Published by a pub.dev verified publisher"){kind=small}
Hi @KoriIku ! Thanks for providing some helpful extra clarification for the docs on data collections!
Can you please remove the translation file in this PR? When content changes are proposed to the English site, the PR should contain only a change to the English file, which is treated as the source file for our i18n tracking system.
Only after the English is reviewed, edited, and approved do we consider changes ready for translation. Even small changes to the English site are often revised before publishing, so we ask that you not worry about submitting a translation at the same time, to save you time, and because we have a system for translations already! :raised_hands:
Okay, thank you for your guidance. I have removed the modifications to the translated document.
Hi @KoriIku ! I'm so sorry that it took me so long to get back to this one! Thank you for your patience!
We always prefer to not add extra notes/tips/cautions etc. for regular information that should be part of the text or instruction, so I was waiting until I could look at the page more closely and figure out how to best add this content inside the regular text.
See what you think of this instead! I think this makes the properties clear, and also links back to where on the page the properties are listed out, so that someone can find for themselves where it's mentioned that data collections use id. Are you happy with this change instead?