Merge pull request #146 from yashsinghcodes/quick-fixes-when-pr

[fix]: returning the same apps for the user
Shuffle · Feb 6, 2025 · a40f632 · a40f632
2 parents 384acea + 520a7fb
commit a40f632
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/db-connector.go b/db-connector.go
@@ -6971,6 +6971,7 @@ func GetUserApps(ctx context.Context, userId string) ([]WorkflowApp, error) {
 				for {
 					innerApp := WorkflowApp{}
 					_, err = it.Next(&innerApp)
+					alreadyExists := false
 					//log.Printf("Got app: %s (%s)", innerApp.Name, innerApp.ID)
 					cnt += 1
 					if cnt > maxAmount {
@@ -7001,8 +7002,16 @@ func GetUserApps(ctx context.Context, userId string) ([]WorkflowApp, error) {
 						continue
 					}
 
-					userApps = append(userApps, innerApp)
+					// Not sure if it actually make the API slower
+					for _, app := range userApps {
+						if app.ID == innerApp.ID {
+							alreadyExists = true
+						}
+					}
 
+					if !alreadyExists {
+						userApps = append(userApps, innerApp)
+					}
 				}
 
 				if err != nil {