ForumMagnum / ForumMagnum / 29823446608

48%

master: 47%

Build Type

Pull #12663

github

Committed by claude

Commit Message

Store multiple-choice poll votes like the slider (unify storage)

Match the ea-forum simplification: multiple-choice poll votes are stored
at the top level of publicData keyed by userId, exactly like the slider,
instead of nested under a separate `publicData.votes` object. Since the
MC poll feature is unmerged there is no data to migrate.

- ForumEventsRepo: drop getMcUserVote/addMcVote/removeMcVote; MC voting
  reuses the slider's getUserVote/addVote/removeVote. setMcPollOptions
  becomes a top-level publicData merge that preserves existing votes.
- AddForumEventMcVote resolver uses addVote/removeVote.
- The voteCount resolver and the poll-notification cron's getVoterUserIds
  now use one shared "keys minus reserved config keys" rule instead of
  branching on poll format.
- getMcPollPublicData gathers per-user votes from the top level; inline
  the single-use getMcPollAnswers helper.
- Update the ForumEventsRepo integration test to the shared vote methods.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017xoYSRMi8i81xf362x5vF5

Pull Request Pull Request #12663: Add multiple-choice polls (available in the post editor)

Coverage Stats

7931 of 10748 branches covered (73.79%)

Branch coverage included in aggregate %.

342 of 916 new or added lines in 14 files covered. (37.34%)

52 existing lines in 6 files now uncovered.

177179 of 371971 relevant lines covered (47.63%)

34.08 hits per line