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

Verify captcha after checking for existing review #488

Merged
merged 1 commit into from
Oct 16, 2024

Conversation

js0mmer
Copy link
Member

@js0mmer js0mmer commented Oct 16, 2024

Description

An issue would occur if you try to review a course/professor you've already reviewed and it get rejected, then change to one you haven't reviewed, the captcha token would be rejected. This fixes that.

Screenshots

Steps to verify/test this change:

  • Verify changes work as expected on staging instance

Final Checks:

  • Verify successful deployment

(optional)

  • Write tests
  • Write documentation

Issues

Closes #

@js0mmer js0mmer temporarily deployed to staging-488 October 16, 2024 20:15 — with GitHub Actions Inactive
@js0mmer js0mmer merged commit 3427f81 into master Oct 16, 2024
2 checks passed
@js0mmer js0mmer deleted the verify-captcha-later branch October 16, 2024 20:25
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.

2 participants