-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
[TypeScript] Add inline documentation for most common components #9234
Conversation
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.
Nice!
filter?: FilterPayload; | ||
|
||
/** | ||
* The filter to apply when calling getList if the filter is empty. |
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.
Aren't those the default values of the always on filter inputs?
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.
yes, but we don't need to go into details here. The link to the doc is enough.
Co-authored-by: Gildas Garcia <1122076+djhi@users.noreply.github.com>
Problem
The analytics for the react-admin docs show that developers mostly visit the pages dedicated to the
<Admin>
,<List>
, and<Datagrid>
components.If they need to read the doc, it means they don't have the necessary information at hand.
Solution
Document the proptypes of these components, so that IntelliSense can provide inline usage and doc link.
This is a limited experiment to see if it facilitates onboarding of new developers.