seladb/StarTrack-js

Pagination limit reached when graphing denoland/deno

Byron opened this issue · 3 comments

Byron commented

When entering https://github.com/denoland/deno into https://seladb.github.io/StarTrack-js it will load the star history nearly to the end until this message appears:

Screenshot 2021-10-11 at 08 01 45

It suggests to check the rel=last response header to handle the issue gracefully. Then all already retrieved results, nearly 100% according to the progress bar, are discarded.

A way to solve this issue might be to…

  • …rely on rel=last as suggested
  • …graph the successfully retrieved star history despite the error, instead of discarding it.

Same issue here for vercel/next.js

@Byron @theMasix I'm really sorry for the late response. You're right, that's a good idea! We can also let the user choose between not displaying anything and displaying the successfully retrieved data. Another option (that I need to check if possible) is to retrieve the latest data, meaning instead of starting from page 1 - start with the minimum page that allows fetching all of the data until the present time.

If you have time to work on this - I'd really appreciate it if you can open a PR. Otherwise, I'll try to make time for it, but I can't guarantee when...

The issue has been fixed in version 3.0.0.

I also added a check for the star count and a warning if couldn't load all stars in this PR: #70. It'll be released soon, as part of version 3.1.0