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

fix: bug in appeal creation #74

Merged
merged 1 commit into from
Sep 20, 2023
Merged

fix: bug in appeal creation #74

merged 1 commit into from
Sep 20, 2023

Conversation

bsushmith
Copy link
Collaborator

@bsushmith bsushmith commented Sep 18, 2023

  • bug: if a policy only has auto approval steps, and the appeal gets auto-approved with last approval step as skipped, then with the current logic the appeal is approved but the grant never gets created. Fix this by adding a check

Closes #75

@bsushmith bsushmith linked an issue Sep 18, 2023 that may be closed by this pull request
core/appeal/service.go Outdated Show resolved Hide resolved
- bug: if a policy only has auto approval steps, and the appeal gets auto-approved, then with the current logic the appeal is approved but the grant never gets created. Fix this by adding a check
@bsushmith bsushmith merged commit 80a35d6 into main Sep 20, 2023
6 checks passed
@bsushmith bsushmith deleted the appeal-create-fix branch September 20, 2023 07:04
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.

create appeal using auto approval policy does not work
2 participants