-
Notifications
You must be signed in to change notification settings - Fork 46
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
5512 Handle feedback #5212
5512 Handle feedback #5212
Conversation
…ion since it doesnt save to the DB and its validated on the frontend (keep legacy pattern);
…ase type; updated unit tests;
…for secondary contact;
…512-story-updates
…actsAndReasonsAction
…tion; remove unused field taxYear;
…ssuingOffice to be consistent with the other fields;
@@ -178,11 +174,6 @@ export class ElectronicPetition extends JoiValidationEntity { | |||
'*': 'Your Petition file size is empty', | |||
'number.max': `Your Petition file size is too big. The maximum file size is ${MAX_FILE_SIZE_MB}MB.`, | |||
}), | |||
petitionRedactionAcknowledgement: joi.boolean().when('petitionType', { |
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.
Is there any harm in leaving this in since it'll be added back with 10339? Not a whole lot of point in validating something we don't save here, but also not a whole lot of point in removing it
@@ -1622,6 +1623,11 @@ export const JUDGE_TITLES = [ | |||
] as const; | |||
export type JudgeTitle = (typeof JUDGE_TITLES)[number]; | |||
|
|||
export const PETITION_TYPES = { |
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.
TY!
No description provided.