fecgov / fecfile-web-app

Other
7 stars 4 forks source link

Link pages to Report Left Side Navigation #350

Closed AureliaKhorsand closed 2 years ago

AureliaKhorsand commented 2 years ago

Purpose

Create the Report Workflow Accordion Side Navigation.

Business Reason

As a Committee Administrator, I will be able to navigate thru the Form3X report workflow. I will be able to move back and forth through the workflow

Dev Notes

This is base on the FEC Pattern library component https://fec-pattern-library.app.cloud.gov/components/detail/side-navigation.html

The angular routes may have to be revisited if information that is needed in the current routes is missing. (e.g. :reportId)

Building Specs

QA Notes

Accordion Navigation

Tier 1: ENTER A TRANSACTION - [ALL CAPS]

Tier 2: REVIEW A REPORT

Section 3: SUBMIT YOUR REPORT

QA Notes

To display the report management menu in the left sidebar, click on "Reports" in the top menu and edit a report.

DEV Notes

null

Design

null

See full ticket and images here: FECFILE-1152

mjtravers commented 2 years ago

@WiseQA The left sidebar navigation is disappearing when a user edits a report transaction. The fix for this is a bit involved with the renaming of page URLs and their interaction with the data to pull the report and transaction IDs. I suggest we open a new ticket to address adding the edit transaction screens so that the "ENTER A TRANSACTION" -> "Add a receipt" menu stays highlighted for those screens.

Also, the memo ticket app#166 is still In Progress so the link for that page in the sidebar menu ("Add a report level memo") has been disabled until it is completed and passes CR. It will not be testable with this ticket.

The issue with the edit transaction screens is being worked on in ticket 415: https://app.zenhub.com/workspaces/fecfile-online-619e578e68408b001c831251/issues/fecgov/fecfile-web-app/415

mjtravers commented 2 years ago

Passed CR with known issues regarding the edit transaction screens and the memo screen. (See comment above.) Sending to QA

WiseQA commented 2 years ago

Per discussion with @mjtravers updated Acceptance Criteria with verbage "(NOT BUILT YET-LEAVE UNLINKED)" for consistency and ease of review what has and has not been built out.

QA Review completed, visual inspection completed.

QA completed moved to Stage Ready.