-
-
Notifications
You must be signed in to change notification settings - Fork 112
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
feat: add options declaration type to chart component #2459
feat: add options declaration type to chart component #2459
Conversation
Features
Bug Fixes
Contributorsyvmunayev, LeandroTorresSicilia Commit-Lint commandsYou can trigger Commit-Lint actions by commenting on this PR:
|
src/components/Chart/index.d.ts
Outdated
@@ -19,6 +20,8 @@ export interface ChartProps extends BaseProps { | |||
maintainAspectRatio?: boolean; | |||
plugins?: Array<object>; | |||
children?: ReactNode; | |||
options?: ChartOptions; | |||
[key: string]: unknown; |
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 think here we only need to add options, the [key: string]: unknown;
is only needed for dataset where you can pass extra props defined by plugins but here I don't think we need this
Visit the preview URL for this PR (updated for commit 37112b4): https://react-rainbow--pr2459-feat-add-options-dec-mziflasm.web.app (expires Sun, 09 Oct 2022 17:45:17 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 |
Code Climate has analyzed commit 37112b4 and detected 0 issues on this pull request. View more on Code Climate. |
fix: #2458
Changes proposed in this PR:
feat: add options declaration type to chart component
I have followed (at least) the PR section of the contributing guide.
@nexxtway/react-rainbow