From c36327c9bf787e84f7fcbc0759d27f1c1a2ebf23 Mon Sep 17 00:00:00 2001
From: uvorbs <yuval@orbs.com>
Date: Wed, 27 Mar 2024 09:42:41 +0200
Subject: [PATCH] getSwapFills return empty when no swaps

---
 service/get_orders_for_user.go        | 2 +-
 transport/rest/get_orders_for_user.go | 6 ++++--
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/service/get_orders_for_user.go b/service/get_orders_for_user.go
index f25024a8..0efd8b6a 100644
--- a/service/get_orders_for_user.go
+++ b/service/get_orders_for_user.go
@@ -53,7 +53,7 @@ func (s *Service) GetSwapFills(ctx context.Context, userId uuid.UUID, symbol mod
 
 	if len(swapIds) == 0 {
 		logctx.Warn(ctx, "user has no resolved swaps", logger.Error(err), logger.String("user_id", userId.String()))
-		return []models.Fill{}, models.ErrNotFound
+		return []models.Fill{}, nil
 	}
 
 	fills := []models.Fill{}
diff --git a/transport/rest/get_orders_for_user.go b/transport/rest/get_orders_for_user.go
index f6361df7..cc997150 100644
--- a/transport/rest/get_orders_for_user.go
+++ b/transport/rest/get_orders_for_user.go
@@ -102,12 +102,14 @@ func (h *Handler) GetSwapFills(w http.ResponseWriter, r *http.Request) {
 	fills, err := h.svc.GetSwapFills(r.Context(), user.Id, symbol, startAt, endAt)
 	if err != nil {
 		logctx.Error(r.Context(), "failed GetSwapFills", logger.Error(err), logger.String("userId", user.Id.String()))
-		if err == models.ErrMaxRecExceeded {
+		switch err {
+		case models.ErrMaxRecExceeded:
 			// narrow down the time range, 256 exceeded
 			restutils.WriteJSONError(ctx, w, http.StatusRequestEntityTooLarge, err.Error())
-		} else {
+		default:
 			restutils.WriteJSONError(ctx, w, http.StatusInternalServerError, "Error getting swaps. Try again later")
 		}
+		return
 	}
 	jsonData, err := json.Marshal(fills)
 	if err != nil {