Skip to content
This repository has been archived by the owner on Jan 16, 2022. It is now read-only.

Issue 706 cached packages #35

Closed

Conversation

artemdudkin
Copy link

Type:

PoC version for task "Pages for npm cached packages"
(verdaccio/verdaccio#706)

Description:

New page for list of cached packages
verdaccio-pr

@@ -73,6 +73,7 @@
"react-hot-loader": "4.7.1",
"react-router": "4.3.1",
"react-router-dom": "4.3.1",
"react-virtualized": "9.21.0",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs an open discussion, pagination vs infinite scroll. cc: @ayusharma

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cc: @verdaccio/maintainers @verdaccio/collaborators

Copy link
Author

@artemdudkin artemdudkin Apr 25, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(1) Right now it gets all data at one request, i.e. without pagination, and it is 18kb response for 3k packages at my local repo (and then it was rendered by List component that renders only visible items). It looks ok for me to do it this way now, but if someone gets 10 times more packages with 10 versions for any package then it can be over 1Mb, and this is not good.

(2) If pagination arrives then we need "search" endpoint, I guess?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(1) Agree.
(2) Yes.

I'd go personally with (2), but I'd wait to hear more opinions (hopefully arrive some).

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could stream JSON using Oboe.js or another solution.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you have any example that we can take a look such implementation?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd love to have pagination in verdaccio ✋

@artemdudkin
Copy link
Author

Any thoughts?

@juanpicado juanpicado changed the base branch from 4.x-master to master August 4, 2019 09:43
@juanpicado juanpicado mentioned this pull request Jun 28, 2020
@juanpicado
Copy link
Member

I close this one because we use already react-virtualized

@juanpicado juanpicado closed this Oct 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants