darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2025-12-20 23:41:17 -05:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #822 from GSA/notify-admin-1148

Browse Source 
make sure original_file_name is stored as part of job (notify-admin-1…
This commit is contained in:
Carlo Costino
2024-03-04 14:21:36 -05:00
committed by GitHub
parent 18dddf4043 689d82280b
commit 3db6462a82
1 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/job/rest.py
View File

@@ -159,6 +159,7 @@ def create_job(service_id):
raise InvalidRequest("Create job is not allowed: service is inactive ", 403) raise InvalidRequest("Create job is not allowed: service is inactive ", 403)
data = request.get_json() data = request.get_json()
original_file_name = data.get("original_file_name")
data.update({"service": service_id}) data.update({"service": service_id})
try: try:
data.update(**get_job_metadata_from_s3(service_id, data["id"])) data.update(**get_job_metadata_from_s3(service_id, data["id"]))
@@ -181,6 +182,9 @@ def create_job(service_id):
data.update({"template_version": template.version}) data.update({"template_version": template.version})
job = job_schema.load(data) job = job_schema.load(data)
# See admin #1148, for whatever reason schema loading doesn't load this
if original_file_name is not None:
job.original_file_name = original_file_name
if job.scheduled_for: if job.scheduled_for:
job.job_status = JobStatus.SCHEDULED job.job_status = JobStatus.SCHEDULED