MushroomObserver / mushroom-observer / 20584838782

95%

main: 95%

Build Type

Pull #3659

github

Committed by mo-nathan

Commit Message

Fix regression in lightbox caption sync after marking as reviewed

The bind(this) call was binding the function to the controller instance,
but Turbo expects 'this' to be the turbo-stream element when calling
custom stream actions. This caused this.getAttribute("obs-id") and
this.templateElement to fail.

Removed bind and inlined the function to allow 'this' to be the
turbo-stream element when Turbo invokes the custom action. This fixes
the regression where marking an observation as reviewed and then
entering lightbox mode would show the checkbox as unchecked.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Pull Request Pull Request #3659: Refactor reviewed state logic and add test coverage

Run Details

17 of 18 new or added lines in 4 files covered. (94.44%)

7 existing lines in 3 files now uncovered.

31440 of 33004 relevant lines covered (95.26%)

713.74 hits per line