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(PasswordInput): add typescript types to PasswordInput #13311

Merged

Conversation

shryoo-ibm
Copy link
Contributor

@shryoo-ibm shryoo-ibm commented Mar 8, 2023

Closes #12550

Part of #12513, Adding TypeScript types to carbon react components

Changelog

Changed

  • Adds type declarations to the props of the PasswordInput component

Testing / Reviewing

Types can be tested by making a component and trying out the different property values.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 8, 2023

DCO Assistant Lite bot All contributors have signed the DCO.

@netlify
Copy link

netlify bot commented Mar 8, 2023

Deploy Preview for carbon-components-react ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 817ac56
🔍 Latest deploy log https://app.netlify.com/sites/carbon-components-react/deploys/6425c92c96926e0008fab098
😎 Deploy Preview https://deploy-preview-13311--carbon-components-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 settings.

@netlify
Copy link

netlify bot commented Mar 8, 2023

Deploy Preview for carbon-elements ready!

Name Link
🔨 Latest commit 817ac56
🔍 Latest deploy log https://app.netlify.com/sites/carbon-elements/deploys/6425c92c9945d500081a3435
😎 Deploy Preview https://deploy-preview-13311--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 settings.

@shryoo-ibm shryoo-ibm changed the title refactor: add types to props in PasswordInput feat(PasswordInput): add typescript types Mar 8, 2023
@shryoo-ibm
Copy link
Contributor Author

I have read the DCO document and I hereby sign the DCO.

@shryoo-ibm
Copy link
Contributor Author

recheck

@shryoo-ibm shryoo-ibm force-pushed the add-types-password-input branch from f47a668 to a4416f2 Compare March 8, 2023 19:24
@shryoo-ibm shryoo-ibm changed the title feat(PasswordInput): add typescript types feat(PasswordInput): add typescript types to PasswordInput Mar 8, 2023
@shryoo-ibm shryoo-ibm marked this pull request as ready for review March 8, 2023 21:45
@shryoo-ibm shryoo-ibm requested a review from a team as a code owner March 8, 2023 21:45
Copy link
Member

@tay1orjones tay1orjones left a comment

Choose a reason for hiding this comment

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

Just one suggestion, otherwise I think this looks good!

packages/react/src/components/TextInput/PasswordInput.tsx Outdated Show resolved Hide resolved
@shryoo-ibm shryoo-ibm requested review from tay1orjones and removed request for alisonjoseph March 16, 2023 21:22
Copy link
Member

@tay1orjones tay1orjones left a comment

Choose a reason for hiding this comment

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

Looks great, thanks! 🎉

@tay1orjones
Copy link
Member

@shryoo-ibm looks like there's a minor conflict to resolve - just regen the allcontributors probably

@shryoo-ibm shryoo-ibm requested a review from tay1orjones March 17, 2023 17:15
@aledavila
Copy link
Contributor

bump @shryoo-ibm looks like you have some minor conflicts to fix :) then we can review thanks!

@shryoo-ibm
Copy link
Contributor Author

bump @shryoo-ibm looks like you have some minor conflicts to fix :) then we can review thanks!

Ah okay, thanks!

Copy link
Member

@tay1orjones tay1orjones left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍 thanks!

@kodiakhq kodiakhq bot merged commit 507ed8e into carbon-design-system:main Mar 30, 2023
tw15egan pushed a commit to tw15egan/carbon that referenced this pull request Mar 31, 2023
…sign-system#13311)

* refactor: add types to props in PasswordInput

* refactor: make separate excluded attributes type

* docs: contributor information

* fix: remove feature condition and deprecated size

---------

Co-authored-by: Taylor Jones <[email protected]>
Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
@carbon-bot
Copy link
Contributor

Hey there! v11.26.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
Development

Successfully merging this pull request may close these issues.

Add TypeScript types to PasswordInput
6 participants