Skip to content

Conversation

@pikachu0542
Copy link
Member

What

Adds information on the results screen of a poll that shows the quorum required (as a percentage), the number of eligible voters on that poll, and the number of votes needed to reach quorum.

Why

Being able to see the values for a specific poll is useful for quickly determining if quorum is reached or how far away from quorum a poll is. Additionally, it is useful for if there are potential changes to the number of eligible voters during the time a poll is open, we can see how many votes were needed when a poll was opened.

Test Plan

  • Create a poll without quorum set and verify the displayed info
  • Create a poll with 1/2 quorum and verify the displayed info
    • You may need to input a list of waived users in order to verify the logic works properly, since conditional gatekeep logic doesnt account for everyone being able to vote before 6 weeks
  • Create a poll with 2/3 quorum and verify the displayed info
    • You may need to input a list of waived users in order to verify the logic works properly, since conditional gatekeep logic doesnt account for everyone being able to vote before 6 weeks

Env Vars

No changes

Checklist

  • Tested all changes locally

@pikachu0542 pikachu0542 self-assigned this Feb 3, 2026
@pikachu0542 pikachu0542 added enhancement New feature or request go Pull requests that update go code labels Feb 3, 2026
@pikachu0542 pikachu0542 linked an issue Feb 3, 2026 that may be closed by this pull request
@tallen42
Copy link
Contributor

tallen42 commented Feb 3, 2026

Ran on my machine, looks good.

@tallen42 tallen42 merged commit c7bb0b2 into dev Feb 3, 2026
1 check passed
tallen42 added a commit that referenced this pull request Feb 5, 2026
* Added logging of voting count info when poll closes

* Update log message

* removed log calls, will be displaying that info on results page instead

* Added quorum info to results page

* Made it look nice, trying to display quorum type as percentage

* clean percentages

* Fixed merge conflict

* Made text sizes of info consistent

---------

Co-authored-by: Tyler Allen <tyler@tallen.me>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request go Pull requests that update go code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Logging

2 participants