influxdata / influxdata/clockface
EPIC: Separate "Disabled" from ComponentStatus
- Dominant language
- TypeScript
- Stars
- 46
- Forks
- 18
- Avg merge
- 2d 7h
- Merged PRs (30d)
- 1
Description
`ComponentStatus` seems more like a mirror of `RemoteDataState`, which doesn't have any concept of "disabled".
We have some components that have a separate `disabled` prop as a boolean, which seems to mesh nicely with what's in `ComponentStatus`
Proposal:
- Go through all the components that use `ComponentStatus` and add a separate `disabled` prop
- Remove `Disabled` from `ComponentStatus`
Contributor guide
Research direction
Start by locating every component that uses ComponentStatus and compare its values with RemoteDataState. Done means the affected components handle disabled separately and ComponentStatus no longer includes Disabled, with all affected component behavior checked.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- react, typescript
- Domain
- frontend
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100