Managers can add notes and set status of reports

[jasalisbury]

Description of change

• Review page is now different if the user looking at the page is the approving manager
• Manager's review page includes the user's additional note and allows the manager to leave a manager note
• Manager note added to DB
• Report state is shown in the side nav's "review" section once it leaves "draft" mode
• Policy, service and route added to update status of a report

How to test

1. Pull down, run migrations/seeders
2. Make sure the CURRENT_USER_ID is set to "1"
3. Fill out a complete report. Add "Hermione" as the approving manager
4. Refresh the report, since Hermione is user with the id 1 you should see the approving manager version of the review page.
5. Add a manager note and set the status. Note once you submit the status on the side nav Review and submit section updates
6. Refresh the page to verify the manager notes persists

Issue(s)

• Manager Note for Activity Report HHS/Head-Start-TTADP#138
• Approving manager adding note to activity report HHS/Head-Start-TTADP#231
• Manager update of AR status HHS/Head-Start-TTADP#236
• Manager approves an Activity Report HHS/Head-Start-TTADP#137

Checklist

• Meets issue criteria
• Code tested
• Meets accessibility standards (WCAG 2.1 Levels A, AA)
• Documentation updated
  • API methods
  • Boundary and Data Flow Diagrams
  • Architectural Decision Records written for major infrastructure decisions with the Nygard template
  • OSCAL templates completed when security controls are implemented or modified

[jasalisbury]

API docs aren't working for me for some reason, will look into it.

[dcmcand]

Currently the manager notes are not connected to a particular user. I know that there will only be one approving manager per activity report, but I could see a situation where the approving manager gets changed. Is this a concern? Something to look at post mvp?

[jasalisbury]

Currently the manager notes are not connected to a particular user. I know that there will only be one approving manager per activity report, but I could see a situation where the approving manager gets changed. Is this a concern? Something to look at post mvp?

That is a good question. My first thought is to not worry about it until after the MVP, thoughts @rahearn?

[rahearn]

My first thought is to not worry about it until after the MVP

Yeah, it's not ideal but I think we can punt this until we implement multiple approvals.

[dcmcand]

Looks good