diff --git a/internal/app/backend/backend_service.go b/internal/app/backend/backend_service.go index 22d73cf3c..92d4cdee6 100644 --- a/internal/app/backend/backend_service.go +++ b/internal/app/backend/backend_service.go @@ -447,9 +447,7 @@ func doAssignTickets(ctx context.Context, req *pb.AssignTicketsRequest, store st } if err = store.DeleteTicketsFromPendingRelease(ctx, ids); err != nil { - logger.WithFields(logrus.Fields{ - "ticket_ids": ids, - }).Error(err) + logger.WithError(err).WithField("ticket_ids", ids).Error("failed to delete tickets from pending release after updating assignment") } return resp, nil diff --git a/internal/app/frontend/frontend_service.go b/internal/app/frontend/frontend_service.go index 6191e4da2..05f757dce 100644 --- a/internal/app/frontend/frontend_service.go +++ b/internal/app/frontend/frontend_service.go @@ -374,6 +374,10 @@ func (s *frontendService) AcknowledgeBackfill(ctx context.Context, req *pb.Ackno } } + if err = s.store.DeleteTicketsFromPendingRelease(ctx, associatedTickets); err != nil { + logger.WithError(err).WithField("ticket_ids", associatedTickets).Error("failed to delete tickets from pending release after updating assignment during backfill ack") + } + // Remove all tickets associated with backfill, because unassigned tickets are not found only err = s.store.UpdateBackfill(ctx, bf, []string{}) if err != nil {