diff --git a/validation/src/check_mapfix.rs b/validation/src/check_mapfix.rs index de95811..20513b5 100644 --- a/validation/src/check_mapfix.rs +++ b/validation/src/check_mapfix.rs @@ -37,14 +37,18 @@ impl crate::message_handler::MessageHandler{ ErrorMessage:report, } ).await.map_err(Error::ApiActionMapfixCheck)?, - // TODO: report the error // update the mapfix model status to request changes - Err(e)=>self.api.action_mapfix_request_changes( - submissions_api::types::ActionMapfixRequestChangesRequest{ - MapfixID:mapfix_id, - ErrorMessage:e.to_string(), - } - ).await.map_err(Error::ApiActionMapfixCheck)?, + Err(e)=>{ + // log error + println!("[check_mapfix] Error: {e}"); + + self.api.action_mapfix_request_changes( + submissions_api::types::ActionMapfixRequestChangesRequest{ + MapfixID:mapfix_id, + ErrorMessage:e.to_string(), + } + ).await.map_err(Error::ApiActionMapfixCheck)?; + }, } Ok(()) diff --git a/validation/src/check_submission.rs b/validation/src/check_submission.rs index 4140539..4e0842f 100644 --- a/validation/src/check_submission.rs +++ b/validation/src/check_submission.rs @@ -38,14 +38,18 @@ impl crate::message_handler::MessageHandler{ ErrorMessage:report, } ).await.map_err(Error::ApiActionSubmissionCheck)?, - // TODO: report the error // update the submission model status to request changes - Err(e)=>self.api.action_submission_request_changes( - submissions_api::types::ActionSubmissionRequestChangesRequest{ - SubmissionID:submission_id, - ErrorMessage:e.to_string(), - } - ).await.map_err(Error::ApiActionSubmissionCheck)?, + Err(e)=>{ + // log error + println!("[check_submission] Error: {e}"); + + self.api.action_submission_request_changes( + submissions_api::types::ActionSubmissionRequestChangesRequest{ + SubmissionID:submission_id, + ErrorMessage:e.to_string(), + } + ).await.map_err(Error::ApiActionSubmissionCheck)?; + }, } Ok(()) diff --git a/validation/src/create_mapfix.rs b/validation/src/create_mapfix.rs index 9980c53..0884efc 100644 --- a/validation/src/create_mapfix.rs +++ b/validation/src/create_mapfix.rs @@ -43,6 +43,9 @@ impl crate::message_handler::MessageHandler{ let create_result=self.create_mapfix_inner(create_info).await; if let Err(e)=create_result{ + // log error + println!("[create_mapfix] Error: {e}"); + self.api.action_operation_failed(submissions_api::types::ActionOperationFailedRequest{ OperationID:operation_id, StatusMessage:e.to_string(), diff --git a/validation/src/create_submission.rs b/validation/src/create_submission.rs index 3cfc3cc..56b3bd0 100644 --- a/validation/src/create_submission.rs +++ b/validation/src/create_submission.rs @@ -57,6 +57,9 @@ impl crate::message_handler::MessageHandler{ let create_result=self.create_submission_inner(create_info).await; if let Err(e)=create_result{ + // log error + println!("[create_submission] Error: {e}"); + self.api.action_operation_failed(submissions_api::types::ActionOperationFailedRequest{ OperationID:operation_id, StatusMessage:e.to_string(), diff --git a/validation/src/validate_mapfix.rs b/validation/src/validate_mapfix.rs index 56f29bf..e08197d 100644 --- a/validation/src/validate_mapfix.rs +++ b/validation/src/validate_mapfix.rs @@ -26,6 +26,9 @@ impl crate::message_handler::MessageHandler{ ).await.map_err(Error::ApiActionMapfixValidate)?; }, Err(e)=>{ + // log error + println!("[validate_mapfix] Error: {e}"); + // update the mapfix model status to accepted self.api.action_mapfix_accepted(submissions_api::types::ActionMapfixAcceptedRequest{ MapfixID:mapfix_id, diff --git a/validation/src/validate_submission.rs b/validation/src/validate_submission.rs index c60a88b..5bf97b3 100644 --- a/validation/src/validate_submission.rs +++ b/validation/src/validate_submission.rs @@ -26,6 +26,9 @@ impl crate::message_handler::MessageHandler{ ).await.map_err(Error::ApiActionSubmissionValidate)?; }, Err(e)=>{ + // log error + println!("[validate_submission] Error: {e}"); + // update the submission model status to accepted self.api.action_submission_accepted(submissions_api::types::ActionSubmissionAcceptedRequest{ SubmissionID:submission_id,