nasa / earthdata-download

Download your Earth science data with only one click
https://nasa.github.io/earthdata-download/
Other
22 stars 6 forks source link

EDD-26: Adds file list header, refactors download list and files list #21

Closed macrouch closed 1 year ago

macrouch commented 1 year ago

Overview

What is the feature?

Adds a file list header. Refactored download list and file list to both utilize the ListPage component. Implemented react-window and refactored progress reporting to address performance issues in files list

What areas of the application does this impact?

Download list, files list

Testing

Reproduction steps

Start a new download, ensure the download list still works as expected. Click the download to view the files list

Checklist

codecov-commenter commented 1 year ago

Codecov Report

Merging #21 (e8aea06) into main (397a6fd) will increase coverage by 6.66%. The diff coverage is 92.98%.

@@            Coverage Diff             @@
##             main      #21      +/-   ##
==========================================
+ Coverage   76.42%   83.08%   +6.66%     
==========================================
  Files          78       91      +13     
  Lines        1510     1632     +122     
  Branches      349      370      +21     
==========================================
+ Hits         1154     1356     +202     
+ Misses        355      276      -79     
+ Partials        1        0       -1     
Files Changed Coverage Δ
src/app/components/Button/Button.jsx 100.00% <ø> (ø)
src/app/components/Progress/Progress.jsx 100.00% <ø> (ø)
src/app/components/Toast/Toast.jsx 88.23% <ø> (ø)
src/app/constants/humanizedDownloadStates.js 83.33% <ø> (-16.67%) :arrow_down:
.../dialogs/InitializeDownload/InitializeDownload.jsx 100.00% <ø> (ø)
src/app/utils/createVariantClassName.js 100.00% <ø> (ø)
src/main/eventHandlers/cancelDownloadItem.ts 100.00% <ø> (ø)
src/main/preload.ts 0.00% <0.00%> (ø)
src/main/utils/setupEventListeners.ts 0.00% <0.00%> (ø)
src/app/components/ListPage/ListPage.jsx 72.22% <50.00%> (+0.22%) :arrow_up:
... and 26 more