-
Notifications
You must be signed in to change notification settings - Fork 16
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have you poked around https://github.com/jenkins-infra/plugin-site ? There's pagination there that can probably be copied. Returning all the plugins ever is going to take awhile. I figured that a lot of this application could mirror the pagination and filtering done by that project.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's resolve the merge conflict after merging in the other PRs and then get this one merged so anyone can do a full run of the system.
@kwhetstone I have to maybe restrict the plugins to 50 maybe so that I can test it out. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@baymac want to review?
This PR connects the plugin api with the front end to load the plugin cards dynamically.
I have not included the description inside it because some of the descriptions has some href links which were not rendered properly. Another issue is that the plugin api takes
48 seconds
to come back with a response which is a bit concerning. Also the entire plugin list of 1600 plugins is displayed on a single screen which I guess is not right and we need to implement scrolling.