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.

Sign up for GitHub

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

Jump to bottom

BugFix: Deny Reason does not get cleared #208

Merged

Xyphyn merged 1 commit into Xyphyn:main from Neshura87:bugfix-deny-modal

Nov 29, 2023

Contributor

Neshura87 commented Nov 29, 2023

Steps to reproduce Bug:

Start Denial Process
Enter any deny reason
cancel Denial
approve request instead

Result: Approved Application has displayed deny reason.

This fixes that by clearing the denyReason var before cancelling out of the review function


          Fix: Deny Reason does not get cleared

9624fe4

vercel bot commented Nov 29, 2023

Someone is attempting to deploy a commit to a Personal Account owned by @Xyphyn on Vercel.

@Xyphyn first needs to authorize it.

Neshura87 marked this pull request as ready for review

November 29, 2023 18:44

Xyphyn merged commit 72f088f into Xyphyn:main

1 check passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet