-
Notifications
You must be signed in to change notification settings - Fork 992
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
Fix download collection button in history panel. #13690
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.
Sorry I broke this 😔, that typo and having the legacy routes as fallback made me completely miss it.
I'm far from my laptop until next Tuesday but I managed to write a review using the GitHub app on my phone, it's awesome!
Thank you so much for fixing it!
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.
These proposed changes should implement @davelopez's suggestion, and fix a failing BioBlend test for me.
Co-authored-by: Nicola Soranzo <nicola.soranzo@gmail.com>
Co-authored-by: Nicola Soranzo <nicola.soranzo@gmail.com>
Downloading collections from the classic history panel seems broken in dev. I assume the FastAPI endpoint was broken-ish and then we dropped the WSGI backups and the functionality broke.
I needed to drop the tags in order to not get a 404 and I needed a signature with history_id in it in order to not get a validation error. So this is probably not the best way - just the fastest? @davelopez any ideas?
How to test the changes?
(Select all options that apply)
License