Seemed to me there was no way to browse all the repositories being seeded on a node. On the massive public nodes this is not that useful a feature since there are too many and too jumbled an assortment, but for a smaller personal node, this feature might be valuable to make the explorer’s contents a bit easier to explore.
Went with the most unintrusive implementation of this idea for now; just enough that it “does the job” (you can access this view, and paginate the results) but does not have any of the bells and whistles it should have (search/filtering, sorting, page query parameters support).
As part of this work, since the repos section gets much more complex, I extracted
the repos part of the page into its own component. Otherwise, it is just very simply
allowing the parameters to the fetchRepoInfos call to be manipulated from the page.
I’ve deployed this version of the explorer to my own server, to see the results live.
https://app.seed.eldridge.cam/nodes/seed.eldridge.cam/
Seemed to me there was no way to browse all the repositories being seeded on a node. On the massive public nodes this is not that useful a feature since there are too many and too jumbled an assortment, but for a smaller personal node, this feature might be valuable to make the explorer’s contents a bit easier to explore.
Went with the most unintrusive implementation of this idea for now; just enough that it “does the job” (you can access this view, and paginate the results) but does not have any of the bells and whistles it should have (search/filtering, sorting, page query parameters support).
As part of this work, since the repos section gets much more complex, I extracted
the repos part of the page into its own component. Otherwise, it is just very simply
allowing the parameters to the fetchRepoInfos call to be manipulated from the page.
I’ve deployed this version of the explorer to my own server, to see the results live.
https://app.seed.eldridge.cam/nodes/seed.eldridge.cam/
Adjusted appearance of the “current page” indicator.
Nice work!
One thing I’m on the fence about is that most views encode subviews in URL params, so you can link to any of the pages you’ve selected. For example: https://app.radicle.xyz/nodes/iris.radicle.xyz/rad:z3gqcJUoA1n9HaHKufZs5FCSGazv5/patches/0433dfd5250e3b8551579b19b0b70827a5ec7a9d?tab=changes
I think it would require a bigger rewrite of this patch to pull that off. Additionally we’d have to put in some thought of what structure we want, so maybe the way you do it, is the most unintrusive way to pull it off and we can do it this way initially.
There’s a bug somewhere, chose iris.radicle.xyz as the node, then:
- click browse all
- go to page 5 (or any other)
- click See pinned repos
- it says “This node doesn’t have any pinned repositories.” although the node does have pinned repos
Fixed the mentioned bug and mobile layout
I agree that this should get encoded into the URL, it wouldn’t be too much
trouble to add now if that would be preferred. Could use ?tab=all&page=3,
somewhat consistent with the one you’ve linked. Or that can all come as a
future update, along with searching functionality, since this is not that
useful a feature for any permissive node until then anyway.
I agree that this should get encoded into the URL, it wouldn’t be too much trouble to add now if that would be preferred. Could use ?tab=all&page=3, somewhat consistent with the one you’ve linked. Or that can all come as a future update, along with searching functionality, since this is not that useful a feature for any permissive node until then anyway.
Show all repos if there are no pinned repos
How would this handle backwards compatibility? Like if I browsed a node that hasn’t upgraded yet to the latest httpd api?
Left align pagination on mobile and smaller screens
I tweaked the alignment on mobile and smaller screens. Thanks for the contribution!
Rebase