-
Notifications
You must be signed in to change notification settings - Fork 3
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
MCR-3903: Rate name does not include submission id #2444
MCR-3903: Rate name does not include submission id #2444
Conversation
…on-id # Conflicts: # services/app-web/src/testHelpers/apolloMocks/contractPackageDataMock.ts
} = rateFormData | ||
const pNames = programNames(statePrograms, rateProgramIDs) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pearl-truss Quick question about this. I assume that rateFormData. rateProgramIDs would contain either not a program
or program
and never both. Is that correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@JasonLin0991 I'm sorry I'm just seeing this. I believe it should include both because some states will have a mixture
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Summary
MCR-3903
MCR-[state]-[not a programs || programs]-[start date]-[end date]-[rate type]-[certification date]
for exampleMCR-VA-PROGRAM1-PROGRAM2-20240501-20250501-CERTIFICATION-20240706
.Related issues
Screenshots
The top rate is with the new rate name format. The bottom, an old rate, is a linked rate with the old rate name format retained.
Test cases covered
generateRateCertificationName.test.ts
'Amendment rate test'
'New rate test'
'New rate with no dates'
'Amendment rate with no dates'
'New rate with incomplete dates'
'Incomplete amendment rate dates'
'Rate type not specified'
'No rate programs'
QA guidance