-
-
Notifications
You must be signed in to change notification settings - Fork 300
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fetch more than 30 forks #13
Comments
@Newgap thats a good point. Would be open for a PR from community as I have couple of busy weeks. |
I will work on this. |
all yours @Syed-Umair |
@techgaun, Do you need me to load specific amount of forks data like 100 or all that is available. |
@Syed-Umair I think the best would be to follow the guidelines and put lazy loading or page numbering at the bottom of the page: You should be able to use the Link headers information to do this |
Ok, I will do that. |
Don't forget that if we change the sorting, you will need to refetch the data from GitHub with that new sorting. Or, you can fetch all data in memory and sort that instead. |
@Newgap agreed. I am however not sure if we really want to fetch all data. For examle, |
Can we set the number of forks to be fetched via UI instead of using a max of 100 like in #14? I usually use this tool to find the fork with the latest commit, which is impossible or at least very luck-based with repos with more than 100 forks. |
@ModischFabrications fair point. That's probably a good idea. Will try to get that in soon |
Forks from GitHub are paged 30 at a time. It would be nice to fetch them all for large projects (many forks).
The text was updated successfully, but these errors were encountered: