Skip to content
This repository has been archived by the owner on Sep 3, 2020. It is now read-only.

ADM Approval Workflow #268

Closed
wants to merge 6 commits into from
Closed

ADM Approval Workflow #268

wants to merge 6 commits into from

Conversation

warrenspe
Copy link

feat(opportunity): Adds a workflow for ADMs to approve opportunities

  • Makes opportunities default to Draft state rather than Pending
  • Removes the Publish button from the opportunity page in favor of a Submit for Approval button
  • Adds an Approval Email page, directed to by the Submit for Approval button
  • Caches adm/divisional/branch emails on the user model; pulls them to populate the approval emails
  • Sends emails to the listed addresses and updates the opportunity when the approval page is submitted

Fixes #236

* Makes opportunities default to the Draft state instead of Pending
* Renames the canPublish flag to canSubmit
  Also fixes the logic used to set some unused flags canPublish flags
* Updates warnings on the opportunity view page to reflect the next
  step being submission for approval rather than publishing
* Removes the Publish link from the opportunity view page
* Adds a Submit for Approval button to the opportunity view page
* opportunityAdmEmail, opportunityBranchEmail, opportunityDivisionalEmail
* Adds an opportunity approval state/form
* Adds a controller to handle opportunity approval forms
* Adds an endpoint through the notification service to send
  ad hoc emails from the client side
* Adds a (rough) email template for sending approval requests to the ADM
@scchapma scchapma closed this Apr 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants