Skip to content

Add emoji reactions to notes during discuss - #38

Merged
ellgreen merged 1 commit into
mainfrom
note-reactions
Sep 12, 2026
Merged

ellgreen merged 1 commit into
mainfrom
note-reactions

Conversation

@ellgreen

Copy link
Copy Markdown
Owner

Summary

  • Adds a fixed set of six emoji reactions (👍 🎉 😂 😮 👀 ❤️) that can be toggled on any note, scoped to the discuss stage only — matching when votes themselves become visible, so nothing earlier in the flow (brainstorm/group/vote) is influenced by visible reaction counts.
  • Reactions broadcast live over the existing socket/broker pattern (mirrors how notes and tasks already work), with a new reactions table, DAL, model, and reaction_toggle broker handler that reuses authoriseNote to reject notes from another retro.
  • Frontend adds a reaction bar + emoji picker on Note, wired up only in the discuss screen so Brainstorm/Vote are unaffected.

Test plan

  • go build ./... && go vet ./... && go test ./...
  • cd ui && pnpm lint && pnpm exec tsc -b && pnpm test && pnpm build
  • Manually verified in the browser: created a retro, advanced it to discuss, reacted to a note (pill appeared live, correctly highlighted), toggled it off, and confirmed the state persists correctly across a reload.

🤖 Generated with Claude Code

Gives the discuss stage a lightweight, fun signal separate from voting:
a fixed set of six emoji can be toggled on any note, broadcast live
over the same socket/broker pattern notes and tasks already use.
Scoped to discuss only, matching when votes themselves become visible,
so nothing earlier in the flow is influenced by visible reaction counts.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@ellgreen
ellgreen merged commit e7dd18f into main Sep 12, 2026
2 checks passed
@ellgreen
ellgreen deleted the note-reactions branch September 12, 2026 16:24
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