Migrate InfoBox components to the new Alert component
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 13.8k
- Forks
- 1.4k
- Avg merge
- 17h 58m
- Merged PRs (30d)
- 5
Description
Problem to solve
Following up from https://github.com/gitpod-io/gitpod/issues/9411, this issue acts as a placeholder for migrating the InfoBox components which could be a bit more tricky, and could require some discussion upfront, given how it's been used in some cases, see https://github.com/gitpod-io/gitpod/issues/8701.
Proposal
Migrate InfoBox (8 component instances) to the new Alert component. ➿
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by comparing components/dashboard/src/components/AlertBox.tsx with components/dashboard/src/components/Alert.tsx, then inspect the eight /components/InfoBox usages identified in the issue. Account for the cases referenced in issues 9411 and 8701, and consider the migration complete when all eight instances use Alert.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- frontend
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100