galley: Use bulk query when getting all feature configs for a team us…

…er (#4325) This used to be specialized, but mistakenly got converted to multiple DB calls here: https://github.com/wireapp/wire-server/pull/4178/files#diff-9daaabfa64d3ada89e121d9f68351f16a0237abf97d8fdc672acde7f9f2fd2ffL197-L207https://github.com/wireapp/wire-server/pull/4178/files#diff-9daaabfa64d3ada89e121d9f68351f16a0237abf97d8fdc672acde7f9f2fd2ffL197-L207
wireapp · Nov 4, 2024 · ce9d00a · ce9d00a
1 parent 2d98df4
commit ce9d00a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/changelog.d/3-bug-fixes/optimize-get-all-features b/changelog.d/3-bug-fixes/optimize-get-all-features
@@ -0,0 +1 @@
+galley: Use bulk query when getting all feature configs for a team user
diff --git a/services/galley/src/Galley/API/Teams/Features/Get.hs b/services/galley/src/Galley/API/Teams/Features/Get.hs
@@ -246,7 +246,9 @@ getAllTeamFeaturesForUser ::
   Sem r AllTeamFeatures
 getAllTeamFeaturesForUser uid = do
   mTid <- getTeamAndCheckMembership uid
-  hsequence' $ hcpure (Proxy @(GetAllTeamFeaturesForUserConstraints r)) $ Comp $ getFeatureForTeamUser uid mTid
+  case mTid of
+    Nothing -> hsequence' $ hcpure (Proxy @(GetAllTeamFeaturesForUserConstraints r)) $ Comp $ getFeatureForUser uid
+    Just tid -> getAllTeamFeatures tid
 
 getSingleFeatureForUser ::
   forall cfg r.
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		galley: Use bulk query when getting all feature configs for a team user