Skip to content

Fix board watch button not updating after toggle - #3027

Open
tomycostantino wants to merge 1 commit into
basecamp:mainfrom
tomycostantino:fix-board-watch-turbo-update
Open

Fix board watch button not updating after toggle#3027
tomycostantino wants to merge 1 commit into
basecamp:mainfrom
tomycostantino:fix-board-watch-turbo-update

Conversation

@tomycostantino

Copy link
Copy Markdown
Contributor

Summary

  • Watching/stop watching a board left the button stale until a full refresh
  • format.turbo_stream was declared without a template
  • Drop that format and keep the existing HTML response

How to reproduce

  • Open a board
  • Click STOP WATCHIN, or WATCH THIS, button should flip to opposite
  • Confirm avatars/watchers list updates with the toggle

Problem

screenrecording-2026-08-07_11-41-32.mp4

Fix

screenrecording-2026-08-07_11-52-05.mp4

Copilot AI balanced review requested due to automatic review settings August 7, 2026 14:54

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes stale board watch controls by allowing Turbo requests to fall back to the existing HTML response.

Changes:

  • Removes the unsupported Turbo Stream response format.
  • Preserves HTML and JSON responses.

Tip

If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or run gh pr ready --undo.
Click "Ready for review" or run gh pr ready to reengage.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@@ -6,7 +6,6 @@ def update

respond_to do |format|
format.html
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