-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[docs] Add support for API pages of unstable components #6981
Conversation
const projectsWithThisComponent = allProjectsName.filter((projectName) => { | ||
const currenetProject = projects.get(projectName) as Project; | ||
const symbol = currenetProject.exports[reactApi.name]; | ||
reactApi.packages = Array.from(projects.keys()) |
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.
This modification allow to have the correct import with Unstable_
on the api page
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.
You probably meant this modification? 🤔
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.
The whole block
We need to get the symbol (otherwise we have no import at all on the api page) and we need to pass the name of the export (otherwise the import would not have the Unstable_
prefix)
@@ -4,27 +4,91 @@ | |||
|
|||
## Components | |||
|
|||
- [DateCalendar](/x/api/date-pickers/date-calendar/) | |||
### Pickers |
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.
I added the unstable components and ordered everything because it was becoming impossible to read
These are the results for the performance tests:
|
const projectsWithThisComponent = allProjectsName.filter((projectName) => { | ||
const currenetProject = projects.get(projectName) as Project; | ||
const symbol = currenetProject.exports[reactApi.name]; | ||
reactApi.packages = Array.from(projects.keys()) |
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.
You probably meant this modification? 🤔
Fixes #6695