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
New Documentation: Associate options with plugins #7826
Conversation
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit 4523831:
|
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.
Thank you @Osky772 for made this task done 🥇
Two tags have missing @category
. Could you please add this tag?
- dataSchema
- cell
As soon as you resolve it, I'm going to approve and merge.
@@ -740,6 +767,7 @@ export default () => { | |||
* @memberof Options# | |||
* @type {boolean|string|object} | |||
* @default true | |||
* ! TODO @category (there is no fillHandle plugin in the `src/plugins`) |
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.
@@ -294,6 +304,7 @@ export default () => { | |||
* @memberof Options# | |||
* @type {number|number[]|string|string[]|Array<undefined>|Function} | |||
* @default undefined | |||
* @category Core, AutoColumnSize |
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.
As we assumed on slack, it should be associated only with Core.
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.
done
@@ -330,6 +341,7 @@ export default () => { | |||
* @memberof Options# | |||
* @type {number|number[]|string|string[]|Array<undefined>|Function} | |||
* @default undefined | |||
* @category Core, ManualRowResize, AutoRowSize |
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.
As we assumed on slack, it should be associated only with Core.
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.
done
Part of #7624 and #7626
There is only one TODO