Skip to content
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: exporting typescript props for various components - 2 #16448

Merged
merged 3 commits into from
May 21, 2024

Conversation

Gururajj77
Copy link
Contributor

Main issue: #16154

Closes #16381 - TextArea and TextInput
Closes #16383 - ToolTip
Closes #16384 - TileGroup and UnorderedList

Exporting the typescript props for the above mentioned components as part of typescript adoption
(This PR is mentioned as 2, as we have more components where props needs to be exported so there will be more PRs regarding this)

Changelog

Changed

  • exporting interfaces from the .tsx files if not done
  • Importing them to index.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.

Copy link

netlify bot commented May 13, 2024

Deploy Preview for v11-carbon-react ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit a3c4529
🔍 Latest deploy log https://app.netlify.com/sites/v11-carbon-react/deploys/664cb3327893630008afa983
😎 Deploy Preview https://deploy-preview-16448--v11-carbon-react.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@riddhybansal riddhybansal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!

@kenneth-nguyen1
Copy link

Should this PR also include the import and export for TextInputSkeletonProps in packages/react/src/components/TextInput/index.ts? I was looking at this issue #16382 and I assume this PR would also have a fix to include the TextInputSkeletonProps in the index.ts file. Thanks!

@Gururajj77
Copy link
Contributor Author

Should this PR also include the import and export for TextInputSkeletonProps in packages/react/src/components/TextInput/index.ts? I was looking at this issue #16382 and I assume this PR would also have a fix to include the TextInputSkeletonProps in the index.ts file. Thanks!

Have included TextInputSkeletonProps now.

Copy link

netlify bot commented May 21, 2024

Deploy Preview for carbon-elements ready!

Name Link
🔨 Latest commit a3c4529
🔍 Latest deploy log https://app.netlify.com/sites/carbon-elements/deploys/664cb3327423600008a283c8
😎 Deploy Preview https://deploy-preview-16448--carbon-elements.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@tay1orjones tay1orjones added this pull request to the merge queue May 21, 2024
Merged via the queue into carbon-design-system:main with commit 9fc6529 May 21, 2024
21 checks passed
@carbon-automation
Copy link
Contributor

Hey there! v11.58.0 was just released that references this issue/PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants