Skip to content

dev -> main#107

Merged
pikachu0542 merged 2 commits intomainfrom
dev
Apr 9, 2026
Merged

dev -> main#107
pikachu0542 merged 2 commits intomainfrom
dev

Conversation

@pikachu0542
Copy link
Copy Markdown
Member

What

dev -> main

Why

new stuff in dev :yippee:

Test Plan

test on dev, make sure everything works and looks ok

Env Vars

nope

Documentation

none needed

Checklist

  • Tested all changes locally

* Changed all status codes to the http.StatusCode equivalent. Also changed some 500s to 400s where appropriate to indicate user error (#85)

* Started working on navbar

* More progress

* working theme selector

* Eboard vote link appears on all pages

* Started working on navbar

* More progress

* working theme selector

* Rebase with dev

* Finished rebasing with dev

* Added favicon import

* Misc other stuff, navbar is responsive now

* larger nav title

* Quorum text input, create page basically done

* Finished poll creation page

* Ranked choice poll page is done

* Actually use the hidden template

* changed icon

* Fixed vote count on eboard votes

* Added display of total votes submitted to results

* small fixes

* fixed spacing between buttons for eboard vote on mobile

* poll title wrapping in a different place

* Changed underline to bold

* Capitalize description placeholder text

---------

Co-authored-by: Noah Hanford <bigspaceships56@gmail.com>
costowell
costowell previously approved these changes Apr 8, 2026
Copy link
Copy Markdown
Contributor

@costowell costowell left a comment

Choose a reason for hiding this comment

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

hell yeah :shipit:

* Fixed count of votes cast

* Misc fixes
@csh-sonarqube-community
Copy link
Copy Markdown

Failed Quality Gate failed

  • 0.00% Coverage on New Code (is less than 80.00%)

Project ID: ComputerScienceHouse_vote_c5ba863d-30d7-4fa9-97dd-4f2c58f8f5fa

View in SonarQube

Copy link
Copy Markdown
Contributor

@BigSpaceships BigSpaceships left a comment

Choose a reason for hiding this comment

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

👍

@pikachu0542 pikachu0542 merged commit 3fea2c4 into main Apr 9, 2026
3 of 4 checks passed
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.

3 participants