darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-06-08 07:21:13 -04:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #4169 from alphagov/downgrade-existing-letter-error

Browse Source 
Downgrade error log for expected exception
This commit is contained in:
Ben Thorner
2022-03-03 11:27:48 +00:00
committed by GitHub
parent 47cd857388 35e7e5e957
commit 8da8b167c9
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
app/main/views/uploads.py
View File

@@ -270,7 +270,7 @@ def uploaded_letter_preview(service_id, file_id):
except LetterNotFoundError as e:
current_app.logger.warning(e)
# If the file is missing it could be because this is a duplicate
# If the file is missing it's likely because this is a duplicate
# request, the notification already exists and the file has been
# moved to a different bucket. Note that the ID of a precompiled
# notification is always set to the file_id.
@@ -356,9 +356,9 @@ def send_uploaded_letter(service_id, file_id):
try:
metadata = get_letter_metadata(service_id, file_id)
except LetterNotFoundError as e:
current_app.logger.error(e)
current_app.logger.warning(e)
# If the file is missing it could be because this is a duplicate
# If the file is missing it's likely because this is a duplicate
# request, the notification already exists and the file has been
# moved to a different bucket. Note that the ID of a precompiled
# notification is always set to the file_id.