Skip to content

Commit

Permalink
πŸ‘¨β€πŸ’» Added TODO to fix a not handled case
Browse files Browse the repository at this point in the history
  • Loading branch information
lhbelfanti committed Dec 30, 2024
1 parent 6fc73f5 commit 2b60ea3
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions cmd/api/search/criteria/enqueue.go
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,9 @@ func MakeEnqueue(selectCriteriaByID SelectByID, selectExecutionsByStatuses Selec

err = enqueueCriteria(ctx, criteriaDAO.toCriteriaDTO(), executionID)
if err != nil {
// TODO: if the criteria was inserted but the enqueue failed, the criteria should be removed from the DB
// to avoid `AnExecutionOfThisCriteriaIDIsAlreadyEnqueued` if it is needed to enqueue the same criteria again

log.Error(ctx, err.Error())
return FailedToExecuteEnqueueCriteria
}
Expand Down

0 comments on commit 2b60ea3

Please sign in to comment.