fecgov / fecfile-web-app

Other
7 stars 4 forks source link

System to set default report coverage dates to blank in UI for non-standard reports #2169

Open exalate-issue-sync[bot] opened 1 month ago

exalate-issue-sync[bot] commented 1 month ago

Business Reason

As a user, I will see blank dates in the coverage dates for Form 3x when I select a ‘non-standard’ (outside of the standard monthly and quarterly) report type.

Acceptance criteria

Given a ‘Create report’ function to create Form 3X

When I select one of the following report types:

Then the Covering period ‘From’ and ‘To’ dates will default to blank, and can be edited.

QA Notes

Current state is if the user has already selected a Monthly or Quarterly report and those dates auto-filled, they would persist in the date fields if they then selected one of the non-standard reports. This could cause confusion and we need them to blank out.

DEV Notes

null

Design

h2. Wireframe

!12-DAY PRE-GENERAL (12G).png|width=1200,height=895,alt="12-DAY PRE-GENERAL (12G).png"!

!12-DAY PRE-RUNOFF (12R).png|width=1200,height=895,alt="12-DAY PRE-RUNOFF (12R).png"!

See full ticket and images here: FECFILE-1604

exalate-issue-sync[bot] commented 1 month ago

akhorsand commented: [~accountid:712020:6466b08c-50a2-42ee-ad81-cfe79921d3e8] [~accountid:61b0b42cc510bc006b5c03ed] Not even sure this needs a wireframe, but if so, should be quick. Just showing the date fields as blank, maybe?

exalate-issue-sync[bot] commented 1 month ago

Mitchell Jarrett commented: [~accountid:61b0b42cd5986c006a9e1c94] pls review

exalate-issue-sync[bot] commented 6 days ago

Sasha Dresden commented: [~accountid:61b0b42cc510bc006b5c03ed] Got a plan for how to get this to work and it works perfectly except for YE because we have coverage data for YE. I just wanted to double check we want to clear out YE instead of using the YE coverage dates formula we already have in place.

exalate-issue-sync[bot] commented 1 day ago

Sasha Dresden commented: Spoke to Mitchel and confirmed to build to spec