Skip to content

refactor: unify OPDS error notifications - #104

Merged
jdluu merged 1 commit into
mainfrom
refactor/unify-opds-error-notification
Aug 29, 2026
Merged

refactor: unify OPDS error notifications#104
jdluu merged 1 commit into
mainfrom
refactor/unify-opds-error-notification

Conversation

@jdluu

@jdluu jdluu commented Aug 29, 2026

Copy link
Copy Markdown
Owner

Summary

  • Routes offline library deletion and refresh failures through the shared OPDS error notification boundary.
  • Preserves record retention after deletion failures and the existing null return for refresh failures.
  • Adds focused success and failure coverage for both paths.

Scope

Closes #100. Refactor-only and within the active feature freeze. No new user-facing behavior or dependencies.

Validation

  • Focused Vitest: 4 passed
  • Full Vitest: 301 passed
  • TypeScript build: passed
  • Biome check: passed
  • git diff --check: passed

Unrelated generated Android worktree changes were not staged or included.

@jdluu
jdluu merged commit 28694b8 into main Aug 29, 2026
3 checks passed
@jdluu
jdluu deleted the refactor/unify-opds-error-notification branch August 29, 2026 20:06
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.

refactor: unify OPDS error notification boundary

1 participant