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

Send application for review journey #1743

Merged
merged 5 commits into from
Oct 9, 2023
Merged

Send application for review journey #1743

merged 5 commits into from
Oct 9, 2023

Conversation

thomasleese
Copy link
Contributor

This implements the "send application for review" journey where an admin can take an application and ask for an assessor to review the verification. There are still more parts of this journey which are yet to be built, but this is the first stage.

Trello Card

Screenshots

Screenshot 2023-10-06 at 10 49 09
Screenshot 2023-10-06 at 11 02 23
Screenshot 2023-10-06 at 11 05 30

@thomasleese thomasleese requested a review from a team as a code owner October 6, 2023 12:14
@thomasleese thomasleese enabled auto-merge October 6, 2023 13:12
When making a recommendation on the assessment in the verification and
review stage we want to show slightly different content on the page.
This adds a new controller for handling the process of sending the
application to review after the verification stage.
This updates the status page shown for application forms to handle when
the application has been moved to the review stage.
This adds to the status updater service, which moves the application
form to the review stage and status when an admin requests an
application to be sent to review.
This starts a system test for the review recommendation journey. It
can't be fully tested yet as the other parts of the journey aren't built
yet and it's not possible to access this page yet.
@thomasleese thomasleese merged commit 50b2988 into main Oct 9, 2023
12 checks passed
@thomasleese thomasleese deleted the send-for-review branch October 9, 2023 10:01
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