deepfence / deepfence/ThreatMapper

Upgrade react-table from v6 to hook based API of v7

Open
#92 0 comments 0 reactions 2 assignees Claimed by @pandyamarut View on GitHub
enhancement needs-triage
Dominant language
TypeScript
Stars
5.3k
Forks
631
PR merge metrics
No merged PRs in 30d

Description

**Additional context**
Currently we use table component which is using class based API of react-table v6. We should replace it with upgraded version of react-table v7 which uses more modern hook based API and maintained actively. This will make table component more maintainable.

**Describe the solution you'd like**
upgrade the react-table library, replace all the usages of v6 with v7.

**Components/Services**

- UI/Frontend

**UI Components checklist**
- [x] app/scripts/components/settings-view/global-settings/global-settings.js
- [x] app/scripts/components/settings-view/scheduled-jobs/index.js
- [x] app/scripts/components/settings-view/user-audit-logs/index.js
- [x] app/scripts/components/settings-view/user-profile-view/user-list.js
- [x] app/scripts/components/topology-view/donut-details-modal/donut-details-modal.js
- [x] app/scripts/components/vulnerability-view/cve-image-report-row-detail.js
- [x] app/scripts/components/vulnerability-view/cve-image-report.js
- [x] app/scripts/components/vulnerability-view/top-exploitable-vulnerabilities.js
- [x] app/scripts/components/vulnerability-view/registry-scan/credentials-list.js
- [x] app/scripts/components/vulnerability-view/registry-scan/image-list.js
- [x] app/scripts/components/vulnerability-view/vulnerability-table-view/vulnerability-table-v2-view.js
- [ ] topology table components

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.