influxdata / influxdata/clockface

EPIC: Separate "Disabled" from ComponentStatus

Open
#498 1 comment 0 reactions 0 assignees View on GitHub
enhancement Epic tech debt
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.