influxdata / influxdata/giraffe
Visualization: Table: re-port from UI codebase without architectural changes
- Dominant language
- TypeScript
- Stars
- 189
- Forks
- 33
- Avg merge
- 3d 15h
- Merged PRs (30d)
- 4
Description
There are a couple of nagging issues with Table that should be fixed. As one of the fundamental visualizations, there shouldn't be any bugs associated with Table. And even though there may be pending design and product changes for Table, the existing stuff should work.
### Proposal
Instead of porting the Table visualization code from the UI with changes (by attempting to use functional components rather than class components), the port should just be exactly as-is with only minimal and necessary changes. Also, any minor bugs should be fixed. But none of this should be done by changing the underlying code architect such as moving from class components to functional components. The differences between these two implementations may be the reason for the nagging issues.
Specifically, by doing the above, we are consolidating and hoping to address the following:
https://github.com/influxdata/giraffe/issues/788
https://github.com/influxdata/ui/issues/6147
Contributor guide
Research direction
Start by comparing the current Table visualization with its UI codebase counterpart, then review the related reports in issues 788 and UI issue 6147. The work is done when the port matches the existing UI implementation with only necessary changes and the identified Table bugs are fixed without an architectural rewrite.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- data-visualization, frontend
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100