Skip to content

resolve ABB downloads end to end#55

Merged
JeremiahM37 merged 4 commits into
JeremiahM37:mainfrom
sgerner:codex/abb-download-flow
Jun 2, 2026
Merged

resolve ABB downloads end to end#55
JeremiahM37 merged 4 commits into
JeremiahM37:mainfrom
sgerner:codex/abb-download-flow

Conversation

@sgerner

@sgerner sgerner commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Stacked on #53. This PR wires ABB detail URLs through the download flow and routes audiobook torrents to the audiobook category.

What changed:

  • Include abb_url in the browser payload so ABB results can be downloaded from the UI.
  • Resolve ABB detail-page URLs into magnets before adding them to qBittorrent.
  • Infer audiobook media type for audiobook sources so the torrent lands in the audiobook save path.
  • Reuse the same resolution path from request-approval downloads.
  • Add regression coverage for ABB URL resolution and the direct-URL fast path.

Validation:

  • go test ./internal/api -run 'TestQueryBoundedInt|TestResolveTorrentURL'

Review note:

@sgerner sgerner marked this pull request as ready for review June 1, 2026 00:38
@JeremiahM37 JeremiahM37 assigned JeremiahM37 and sgerner and unassigned JeremiahM37 and sgerner Jun 1, 2026
@JeremiahM37

Copy link
Copy Markdown
Owner

Please solve merge conflicts

sgerner and others added 2 commits May 31, 2026 21:49
…flow

# Conflicts:
#	internal/search/audiobookbay.go
#	internal/search/audiobookbay_test.go
@JeremiahM37 JeremiahM37 self-assigned this Jun 2, 2026
@JeremiahM37 JeremiahM37 merged commit 43fe0a4 into JeremiahM37:main Jun 2, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants