Skip to content

PUB-10 Add player mark watched control#32

Open
rovkinmax wants to merge 4 commits into
masterfrom
PUB-10
Open

PUB-10 Add player mark watched control#32
rovkinmax wants to merge 4 commits into
masterfrom
PUB-10

Conversation

@rovkinmax

Copy link
Copy Markdown
Owner

Task Summary

PUB-10 adds a player controls button to mark the current movie or episode as watched, matching the existing watched action behavior from details and episode lists.

Changes include:

  • Resolving current movie/episode watched state in the player interactor.
  • Adding a one-way MarkCurrentWatched player action that calls the existing watching toggle API with status=1 and the current season/episode for series.
  • Syncing current watched state and series episode UI after manual, automatic, and context-menu watched changes.
  • Adding the Просмотрено control after play/pause and before episodes/audio/video controls.
  • Focused interactor and player VM coverage for watched-state resolution, manual mark behavior, no-op paths, failure preservation, and synchronization cases.

Compliance Review

Compliance Review passed with no violations found in the reviewed scope. The implementation does not modify AGENTS.md, .kent/project-contract.md, or authoritative spec/plan files, uses string resources for user-visible UI text, keeps the existing auto-mark path, and follows the existing player VM/interactor/component structure.

Verification

  • ./tools/agentw :app:testDevDebugUnitTest --tests 'com.kino.puber.domain.interactor.player.PlayerInteractorTest' --tests 'com.kino.puber.ui.feature.player.vm.PlayerVMTest'
    • Result: BUILD SUCCESSFUL documented in .todo/player-mark-watched-button/audit-report.md and verification-step-4.md.
  • ./tools/agentw :app:compileDevDebugKotlin
    • Result: BUILD SUCCESSFUL documented in .todo/player-mark-watched-button/audit-report.md and verification-step-4.md.
  • Focused device smoke on locked emulator-5554 with fresh devDebug build/install:
    • Result: passed, Просмотрено visible, /watching/toggle?id=124081&status=1&season=1&video=1 returned 200, no fatal logcat entries.
  • git diff --check HEAD~1..HEAD
    • Result: no whitespace errors.

Known Skipped Checks / Blockers

  • No blockers.
  • Full test suite was not rerun in the ship-PR step; this PR uses the focused build, unit-test, and smoke evidence from the completed workflow artifacts.

rovkinmax added 4 commits July 7, 2026 17:42
# Conflicts:
#	.kent/project-contract.md
#	.kent/workflows/README.md
#	.kent/workflows/puber-bugfix-investigation.json
#	.kent/workflows/puber-dependency-update.json
#	.kent/workflows/puber-feature-delivery.json
#	.kent/workflows/puber-refactor-with-audit.json
#	.kent/workflows/puber-release.json
#	.kent/workflows/puber-smoke-test.json
#	.kent/workflows/puber-test-coverage.json
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.

1 participant