-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
fix: export props for components #16455
fix: export props for components #16455
Conversation
✅ Deploy Preview for v11-carbon-react ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
LGTM 🥳
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.
Looks good!
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.
LGTM, just one comment but it doesn't have to hold up the merge.
import { type GridProps } from './GridTypes'; | ||
export { type GridProps }; |
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.
Could this be one line?
import { type GridProps } from './GridTypes'; | |
export { type GridProps }; | |
export { type GridProps } from './GridTypes'; |
96214a9
Hey there! v11.58.0 was just released that references this issue/PR. |
Closes #16364 Accordion
Closes #16366 Structured List
Closes #16367 Button
Closes #16368 Button
Closes #16369 Column
Closes #16370 Dropdown
Closes #16371 FormLabel
Closes #16372 Grid
Exporting the typescript props for the above mentioned components as part of typescript adoption
(This PR is mentioned as 1, as we have more components where props needs to be exported so there will be more PRs regarding this)
Changelog
Changed
.tsx
files if not doneindex.ts
and exporting them explicitly so that users can import them directly with the component.Testing / Reviewing
No changes to the component, so new testing observations.