feat: Add retry button and book link to downloads page#68
Merged
Conversation
- Failed downloads now show an amber Retry button in the actions column
that re-queues the exact same download (deletes the error task first,
then calls downloadRelease with the stored url/protocol/format)
- Title cell now shows a "View Book" link navigating to /book/{book_id}
so users can reach the book detail page directly from download history
- Make indexer and size_bytes optional in downloadRelease API type
to support retry without needing the full original release metadata
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
When a download failed on the downloads page, there was no way to retry it inline — users had to navigate away, find the book again, and re-search/re-download. There was also no way to click through to the book's detail page from the downloads list.
Changes
Retry failed downloads inline
↺ Retrybutton in the actions columndownload_url,protocol,format, andbook_id— no searching requiredNavigate to book from downloads list
/book/{book_id}API type fix
indexerandsize_bytesoptional ondownloadRelease— they're metadata fields not needed for the actual download, and aren't stored on the task recordBefore / After
/book/42