chore: Merge branch 'dev'

Author: mydearxym

deploy/production/api_server.tar.gz

@@ -181,7 +181,9 @@ defmodule GroupherServer.CMS.Utils.Loader do
   def query({"posts_comments", PostComment}, %{filter: filter, unique: true}) do
     PostComment
     |> join(:inner, [c], a in assoc(c, :author))
-    |> distinct([c, a], a.id)
+    # NOTE:  this distinct not working in production env, so the uniq logic is move to
+    # cut_participators.ex middleware, when the data is large, will cause performace issue
+    # |> distinct([c, a], a.id)
     |> select([c, a], a)
 
     # |> select([c, a], %{

lib/groupher_server/cms/utils/loader.ex

@@ -8,6 +8,8 @@ defmodule GroupherServerWeb.Middleware.CutParticipators do
   # should use WINDOW function
   # see https://github.com/coderplanets/coderplanets_server/issues/16
   #
+  # the Enum.uniq logic is a tmp sulution for distinct comments users, this should be 
+  # in dataloader logic, but the distinct is not working in production env
   """
 
   @behaviour Absinthe.Middleware
@@ -16,11 +18,14 @@ defmodule GroupherServerWeb.Middleware.CutParticipators do
   def call(%{errors: errors} = resolution, _) when length(errors) > 0, do: resolution
 
   def call(%{value: value, arguments: %{filter: %{first: first}} = args} = resolution, _) do
-    %{resolution | value: value |> Enum.reverse() |> Enum.slice(0, first)}
+    %{resolution | value: value |> Enum.uniq() |> Enum.reverse() |> Enum.slice(0, first)}
   end
 
   def call(%{value: value} = resolution, _) do
-    %{resolution | value: value |> Enum.reverse() |> Enum.slice(0, @default_length)}
+    %{
+      resolution
+      | value: value |> Enum.uniq() |> Enum.reverse() |> Enum.slice(0, @default_length)
+    }
   end
 
   def call(resolution, _), do: resolution