Skip to content
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

UX should reflect zenodo pagination only supports first 10k entries #19121

Open
martenson opened this issue Nov 8, 2024 · 4 comments
Open

UX should reflect zenodo pagination only supports first 10k entries #19121

martenson opened this issue Nov 8, 2024 · 4 comments
Labels
area/real-user Problems arising due to "run as real user" kind/bug paper-cut

Comments

@martenson
Copy link
Member

martenson commented Nov 8, 2024

See
https://zenodo.org/api/records?size=25&page=401
https://zenodo.org/api/records?size=100&page=101

Currently we allow going to pages that we have no chance of resolving. This yields an error that you can recover from only by closing the dialogue.

Screenshot 2024-11-08 at 1 35 25 PM

I did not find this described in the docs but seems to be true across invenioRDM instances.

  File "/Users/marten/devel/git/galaxy/lib/galaxy/files/sources/invenio.py", line 473, in _ensure_response_has_expected_status_code
    raise Exception(
Exception: Request to https://zenodo.org/api/records?size=25&page=164378 failed with status code 400: Invalid querystring parameters.
galaxy.webapps.base.api INFO 2024-11-08 13:40:22,156 [pN:main,p:89383,tN:MainThread] MessageException: Problem listing file source path zenodo://zenodo/
@martenson martenson added kind/bug area/real-user Problems arising due to "run as real user" paper-cut labels Nov 8, 2024
@davelopez
Copy link
Contributor

davelopez commented Nov 8, 2024

Does this look like a bug in Zenodo or InvenioRDM API? 🤔

@martenson
Copy link
Member Author

@davelopez to me it feels like a deliberate undocumented choice...but yeah, hard to say

@davelopez
Copy link
Contributor

I don't know. I think I will create an issue upstream to ask for clarification. If the API returns a total number of hits > 10k, you should be able to reach those records.

@davelopez
Copy link
Contributor

Found this: zenodo/zenodo-rdm#217

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/real-user Problems arising due to "run as real user" kind/bug paper-cut
Projects
None yet
Development

No branches or pull requests

2 participants