Skip to content

Commit 4620aae

Browse files
authored
[Comp-762] Warning letter override on issuer change (#683)
1 parent 4436a11 commit 4620aae

File tree

2 files changed

+2
-8
lines changed

2 files changed

+2
-8
lines changed

compliance-api/src/compliance_api/services/warning_letter/warning_letter.py

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -100,13 +100,6 @@ def update_warning_letter(
100100
ServiceUtils.check_requirement_for_enforcement_action(
101101
requirement_ids, EnforcementActionOptionEnum.WARNING_LETTER.value
102102
)
103-
# If the issuing officer id is different from the existing one, regenerate the content
104-
if issuing_officer_id != warning_letter.issuing_officer_id:
105-
update_data["content"] = _create_content(
106-
warning_letter.inspection,
107-
requirement_ids,
108-
issuing_officer_id,
109-
)
110103
with session_scope() as session:
111104
updated_warning_letter = WarningLetterModel.update_warning_letter(
112105
warning_letter_id, update_data, session

compliance-web/src/components/App/Inspections/Profile/Enforcements/WarningLetters/WarningLetterDrawer.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,8 @@ const WarningLetterDrawer: React.FC<WarningLetterDrawerProps> = ({
104104
return formatFormData(warningLetter);
105105
}
106106
return initFormData;
107-
}, [warningLetter, formatFormData]);
107+
// eslint-disable-next-line react-hooks/exhaustive-deps
108+
}, []);
108109

109110
const methods = useForm<EnforcementFormType>({
110111
resolver: yupResolver(warningLetterFormSchema),

0 commit comments

Comments
 (0)