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

Add Typescript types to Checklist #4497

Closed
6 tasks done
Tracked by #4232
matthewgallo opened this issue Mar 6, 2024 · 0 comments · Fixed by #5055
Closed
6 tasks done
Tracked by #4232

Add Typescript types to Checklist #4497

matthewgallo opened this issue Mar 6, 2024 · 0 comments · Fixed by #5055
Assignees
Labels
adopter: automation Issues connected to work that accelerates IBM's automation offerings. adopter: data & ai Issues connected to work that accelerates IBM's Data & AI offerings. adopter: security Issues connected to work that accelerates IBM's Security offerings. adopter: sustainability Issues connected to work that accelerates IBM's sustainability offerings. area: typescript role: dev type: enhancement 💡 New feature or request type: infrastructure 🤖 Issues related to devops, builds, packaging
Milestone

Comments

@matthewgallo
Copy link
Member

matthewgallo commented Mar 6, 2024

As we add Typescript support, want to call out a few helpful tips from the Carbon TypeScript guide that is largely relevant to our library as well.

Changing the extension via git

When changing the file extension — do not simply rename the file. Instead, change the extension via git.

For example,

git mv packages/ibm-products/src/components/ComponentName/ComponentName.js packages/ibm-products/src/components/ComponentName/ComponentName.tsx

Do not add types to internal components or functions

Testing changes

A few different options for testing changes —

  • At the bottom of a file, write a dummy component that uses the component that you’re converting to ensure that you can still pass all appropriate props.
  • Take a storybook example for a component, copy and paste it into the bottom of the .tsx file and validate whether or not it accepts the props as you’ve defined them.

Tasks

Preview Give feedback

Resources

Carbon’s guide on adding component types

@github-project-automation github-project-automation bot moved this to Needs triage 🧐 in Carbon for IBM Products Mar 6, 2024
@elycheea elycheea moved this from Needs triage 🧐 to Community backlog 🚞 in Carbon for IBM Products Mar 7, 2024
@sstrubberg sstrubberg added status: needs triage 🕵️‍♀️ type: infrastructure 🤖 Issues related to devops, builds, packaging role: dev adopter: security Issues connected to work that accelerates IBM's Security offerings. adopter: data & ai Issues connected to work that accelerates IBM's Data & AI offerings. adopter: automation Issues connected to work that accelerates IBM's automation offerings. adopter: sustainability Issues connected to work that accelerates IBM's sustainability offerings. and removed status: needs triage 🕵️‍♀️ labels Mar 28, 2024
@sstrubberg sstrubberg added this to the 2024 Q2 milestone Mar 28, 2024
@sstrubberg sstrubberg moved this from Community backlog 🚞 to Backlog 🌋 in Carbon for IBM Products Mar 28, 2024
@szinta szinta self-assigned this Apr 22, 2024
@szinta szinta moved this from Backlog 🌋 to In progress in Carbon for IBM Products Apr 30, 2024
@szinta szinta linked a pull request May 2, 2024 that will close this issue
@szinta szinta moved this from In progress to Needs review 👋 in Carbon for IBM Products May 2, 2024
@sstrubberg sstrubberg added the type: enhancement 💡 New feature or request label May 14, 2024
@szinta szinta moved this from Needs review 👋 to In review 👀 in Carbon for IBM Products May 15, 2024
@github-project-automation github-project-automation bot moved this from In review 👀 to Done 🚀 in Carbon for IBM Products May 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
adopter: automation Issues connected to work that accelerates IBM's automation offerings. adopter: data & ai Issues connected to work that accelerates IBM's Data & AI offerings. adopter: security Issues connected to work that accelerates IBM's Security offerings. adopter: sustainability Issues connected to work that accelerates IBM's sustainability offerings. area: typescript role: dev type: enhancement 💡 New feature or request type: infrastructure 🤖 Issues related to devops, builds, packaging
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants