-
Notifications
You must be signed in to change notification settings - Fork 1.4k
docs: followup for after initial release #9492
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
Conversation
|
Build successful! 🎉 |
| headings.push(...subHeadings); | ||
| } | ||
| } | ||
| let allTags = (page.exports?.tags || page.exports?.keywords as string[]) || []; |
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.
Should we include page.exports?.relatedPages.map(page => page.title) here? It might be nice to have the hook names accounted for in the docs.
If we do that we probably want to filter out .startsWith('Testing ') so the S2 component testing docs aren't included.
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.
oh yeah that sounds like a good idea, let me do that
|
Build successful! 🎉 |
|
Build successful! 🎉 |
* chore: new docs followup * add headings to search as keywords * fix lint * filter tags * add example to filter
|
Build successful! 🎉 |
apologies for the non-descriptive title but this PR does two things
✅ Pull Request Checklist:
📝 Test Instructions:
🧢 Your Project: