Skip to content

Catalog: package list UI revamp#1047

Merged
nl0 merged 16 commits intomasterfrom
pkg-list-revamp
Aug 10, 2019
Merged

Catalog: package list UI revamp#1047
nl0 merged 16 commits intomasterfrom
pkg-list-revamp

Conversation

@nl0
Copy link
Copy Markdown
Member

@nl0 nl0 commented Jul 30, 2019

TODO

  • filter
  • sort
  • pagination
  • optimize analytics request (use one query for all the packages)

To be merged only after #857

@nl0 nl0 changed the title Catalog: package list UI revamp Catalog: package list UI revamp WIP Jul 30, 2019
@codecov-io
Copy link
Copy Markdown

codecov-io commented Jul 30, 2019

Codecov Report

Merging #1047 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1047   +/-   ##
=======================================
  Coverage   87.58%   87.58%           
=======================================
  Files          26       26           
  Lines        3559     3559           
=======================================
  Hits         3117     3117           
  Misses        442      442

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 22a3384...73047d2. Read the comment docs.

@nl0 nl0 requested a review from akarve August 6, 2019 10:57
@nl0 nl0 changed the title Catalog: package list UI revamp WIP Catalog: package list UI revamp Aug 6, 2019
@akarve
Copy link
Copy Markdown
Member

akarve commented Aug 10, 2019

Can we make search work without or is that bad?

@nl0 nl0 force-pushed the pkg-list-revamp branch from 006cf05 to 73047d2 Compare August 10, 2019 08:23
@nl0 nl0 merged commit 68bb055 into master Aug 10, 2019
@nl0 nl0 deleted the pkg-list-revamp branch August 10, 2019 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants