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

feat: add rejection to contract approval workflow #2175

Merged
merged 1 commit into from
Jul 8, 2022
Merged

Conversation

volkflo
Copy link
Member

@volkflo volkflo commented Jul 8, 2022

Closes: #1679

@volkflo volkflo requested a review from msavy July 8, 2022 11:45
@msavy msavy changed the title feat: add contract rejection to approval workflow feat: add rejection to contract approval workflow Jul 8, 2022
@cypress
Copy link

cypress bot commented Jul 8, 2022



Test summary

17 0 0 0


Run details

Project Apiman
Status Passed
Commit 86c1579 ℹ️
Started Jul 8, 2022 12:42 PM
Ended Jul 8, 2022 12:45 PM
Duration 02:59 💡
OS Linux Ubuntu - 20.04
Browser Electron 100

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

msavy
msavy previously approved these changes Jul 8, 2022
Copy link
Member

@msavy msavy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some small nits to resolve, but overall looks good.

Thanks for collaborating on this!

Added rejection to the approval workflow (including new events and notifications). Users are allowed
to pass in a `rejectionReason` via the UI form that is part of the event/notifications. Rejected
contracts will be automatically deleted once the event was emitted.

Closes: #1679

Co-authored-by: Marc Savy <marc@rhymewithgravy.com>
Co-authored-by: Bastian Gembalczyk <bastian.gembalczyk@scheer-group.com>
@volkflo volkflo force-pushed the contract-rejection branch from f11318c to 95a6a31 Compare July 8, 2022 12:35
@msavy msavy merged commit bf9607a into master Jul 8, 2022
@msavy msavy deleted the contract-rejection branch July 8, 2022 12:58
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.

Allow rejection of contract request, display independent rejected state for client/contract
3 participants