Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Testing Participation Type #1941

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Riolku
Copy link
Contributor

@Riolku Riolku commented May 23, 2022

Depends on #1886. Lots of code was refactored in order for this change to not be horrible to implement.

This is a hefty change, so testing from others would be appreciated.

@Riolku
Copy link
Contributor Author

Riolku commented May 23, 2022

Closes #1670.

@Riolku Riolku force-pushed the testing-participation branch 2 times, most recently from 492a74c to 3d9bafc Compare May 23, 2022 19:49
@Riolku Riolku changed the title Testing participation Add Testing Participation Type May 23, 2022
As suggested, merge the access checks into one. Some tests had to be
changed, because the old checks didn't check the dates.
Choose a scoreboard depending on the participation type. If the contest
hasn't started, choose the testing scoreboard.
@Riolku Riolku marked this pull request as draft May 24, 2022 05:56
@Riolku
Copy link
Contributor Author

Riolku commented May 24, 2022

This PR needs some more unit tests for the new methods. I will do this soon.

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.

1 participant