Skip to content

Fixed question stems with more than 4 answers making students unable to choose past the fourth answer. - #385

Merged
LaserBread merged 2 commits into
OSU-MC:developmentfrom
LaserBread:answer-count-fix
Jan 30, 2026
Merged

LaserBread merged 2 commits into
OSU-MC:developmentfrom
LaserBread:answer-count-fix

Conversation

@LaserBread

Copy link
Copy Markdown
Contributor

Pull Request Template

Description

This change dynamically allocates the array the webpage POSTS to the server for question stem answer options based on the number of possible answers, fixing an issue where students would see an answer past the 4th option just disappear and them getting no credit.

Fixes #372

Checklist:

Before you submit your Pull Request, please make sure you have completed the following tasks:

  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.
  • I have added tests that prove my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes.
  • Any dependent changes have been merged and published in downstream modules.
  • I have tagged my PR with the appropriate label(s).

@LaserBread
LaserBread requested a review from a team as a code owner January 27, 2026 01:32
@LaserBread LaserBread added the bug Something isn't working label Jan 27, 2026

@MigrainePanda MigrainePanda 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.

No issue with useState only running on first render since questions won't change after render.

Looks good!

@github-project-automation github-project-automation Bot moved this from Backlog to In Progress in classroom-polling Jan 27, 2026
@LaserBread
LaserBread added this pull request to the merge queue Jan 30, 2026
Merged via the queue into OSU-MC:development with commit d00ac12 Jan 30, 2026
8 checks passed
@LaserBread
LaserBread deleted the answer-count-fix branch January 30, 2026 00:48
@github-project-automation github-project-automation Bot moved this from In Progress to Done in classroom-polling Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Answering a multiple choice question only sets four answers

2 participants