Skip to content

Automatically add 2 housing points when user passes spring evals#511

Open
pikachu0542 wants to merge 2 commits intodevelopfrom
auto-housing-points
Open

Automatically add 2 housing points when user passes spring evals#511
pikachu0542 wants to merge 2 commits intodevelopfrom
auto-housing-points

Conversation

@pikachu0542
Copy link
Member

@pikachu0542 pikachu0542 commented Mar 16, 2026

What

Automatically adds 2 housing points when passing spring evals

Why

Im not entirely sure how housing points have been added in the past, but i ususally find that they dont get updated for a while (long after the end of the operating session, and i think also after evals presses the new year button). Instead of whatever else was previously done, now it is automatic.

If you pass, 2 points are added when evals presses the pass button on the slideshow. If you fail, housing points are not changed.

Test Plan

Not sure yet, need to figure this out (i dont know how to test this without modifying actual ldap data)

Env Vars

No

Documentation

I dont believe this change requires any documentation updates

Checklist

  • Tested all changes locally

@pikachu0542
Copy link
Member Author

pikachu0542 commented Mar 16, 2026

I have just realized that I have not accounted for if someone gets a conditional at spring evals and then passes when the conditional is evaluated. I will figure out how to handle that and make changes

@pikachu0542 pikachu0542 linked an issue Mar 16, 2026 that may be closed by this pull request
@pikachu0542 pikachu0542 self-assigned this Mar 16, 2026
@csh-sonarqube-community
Copy link

Passed Quality Gate passed

Issues

Measures

Project ID: ComputerScienceHouse_conditional_b4074151-bbfd-4fc3-b87d-10f1c1f625ab

View in SonarQube

@pikachu0542 pikachu0542 marked this pull request as ready for review March 16, 2026 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Housing points should be automatic

1 participant