From 6792ee31276342f4cedec2de64577e13beceed38 Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Tue, 22 Nov 2022 09:22:18 +0100
Subject: [PATCH 1/9] Fix port number type

After d8616899 and https://framagit.org/framasoft/mobilizon/-/merge_requests/1339#note_1894390

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 config/docker.exs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/config/docker.exs b/config/docker.exs
index 72b4e3121..ec5ffb993 100644
--- a/config/docker.exs
+++ b/config/docker.exs
@@ -39,7 +39,7 @@ config :mobilizon, Mobilizon.Storage.Repo,
   password: System.get_env("MOBILIZON_DATABASE_PASSWORD", "password"),
   database: System.get_env("MOBILIZON_DATABASE_DBNAME", "mobilizon"),
   hostname: System.get_env("MOBILIZON_DATABASE_HOST", "postgres"),
-  port: System.get_env("MOBILIZON_DATABASE_PORT", 5432),
+  port: System.get_env("MOBILIZON_DATABASE_PORT", "5432"),
   pool_size: 10
 
 config :mobilizon, Mobilizon.Web.Email.Mailer,

From f9d21c5a38307552ac7f809a5b4adee66c9f0d82 Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Tue, 22 Nov 2022 09:26:39 +0100
Subject: [PATCH 2/9] Fix 3rd-party auth links

Closes #1198

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 js/src/components/User/AuthProvider.vue | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/js/src/components/User/AuthProvider.vue b/js/src/components/User/AuthProvider.vue
index bd586e315..a7a946dbe 100644
--- a/js/src/components/User/AuthProvider.vue
+++ b/js/src/components/User/AuthProvider.vue
@@ -1,5 +1,6 @@
 <template>
   <o-button
+    tag="a"
     outlined
     variant="primary"
     :icon-left="oauthProvider.id"
@@ -9,6 +10,7 @@
     <span>{{ SELECTED_PROVIDERS[oauthProvider.id] }}</span></o-button
   >
   <o-button
+    tag="a"
     outlined
     variant="primary"
     :href="`/auth/${oauthProvider.id}`"

From 7f15f2bad144a53f96a54e846ee04c9f7855330d Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Tue, 22 Nov 2022 09:47:18 +0100
Subject: [PATCH 3/9] Detect Intl.ListFormat availability and add fallback

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 js/src/utils/listFormat.ts | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/js/src/utils/listFormat.ts b/js/src/utils/listFormat.ts
index 47cb20787..c4cb3b508 100644
--- a/js/src/utils/listFormat.ts
+++ b/js/src/utils/listFormat.ts
@@ -8,10 +8,16 @@ const shortDisjunctionFormatter = new Intl.ListFormat(undefined, {
   type: "disjunction",
 });
 
+const listFormatAvailable = typeof Intl?.ListFormat === "function";
+
 export const listShortConjunctionFormatter = (list: Array<string>): string => {
-  return shortConjunctionFormatter.format(list);
+  return listFormatAvailable
+    ? shortConjunctionFormatter.format(list)
+    : list.join(",");
 };
 
 export const listShortDisjunctionFormatter = (list: Array<string>): string => {
-  return shortDisjunctionFormatter.format(list);
+  return listFormatAvailable
+    ? shortDisjunctionFormatter.format(list)
+    : list.join(",");
 };

From 2e9414767c7ad4a0116f89d2ac15850d3a0d7514 Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Tue, 22 Nov 2022 09:48:39 +0100
Subject: [PATCH 4/9] Increase paging timeout to 30

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 lib/mobilizon/storage/page.ex | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/mobilizon/storage/page.ex b/lib/mobilizon/storage/page.ex
index 29f14f651..cc6db6b7c 100644
--- a/lib/mobilizon/storage/page.ex
+++ b/lib/mobilizon/storage/page.ex
@@ -31,7 +31,7 @@ defmodule Mobilizon.Storage.Page do
         fn -> Repo.all(paginate(query, page, limit)) end
       ]
       |> Enum.map(&Task.async/1)
-      |> Enum.map(&Task.await(&1, 15_000))
+      |> Enum.map(&Task.await(&1, 30_000))
 
     %__MODULE__{total: total, elements: elements}
   end

From aeaf22a224d6516d20f603ad9c4d7ead4fc0ff09 Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Tue, 22 Nov 2022 11:32:31 +0100
Subject: [PATCH 5/9] Set correct Content-Type on all AP endpoints

Closes #1210

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 lib/web/controllers/activity_pub_controller.ex | 11 ++++++-----
 lib/web/controllers/page_controller.ex         | 12 ++++++++++++
 2 files changed, 18 insertions(+), 5 deletions(-)

diff --git a/lib/web/controllers/activity_pub_controller.ex b/lib/web/controllers/activity_pub_controller.ex
index 93f132826..2d97e4a55 100644
--- a/lib/web/controllers/activity_pub_controller.ex
+++ b/lib/web/controllers/activity_pub_controller.ex
@@ -85,8 +85,9 @@ defmodule Mobilizon.Web.ActivityPubController do
         actor = Map.get(conn.assigns, :actor)
 
         if actor_applicant_group_member?(group, actor) do
-          json(
-            conn,
+          conn
+          |> put_resp_content_type("application/activity+json")
+          |> json(
             ActorView.render("member.json", %{
               member: member,
               actor_applicant: actor
@@ -168,7 +169,7 @@ defmodule Mobilizon.Web.ActivityPubController do
   def relay(conn, _params) do
     with {status, %Actor{} = actor} when status in [:commit, :ok] <- Cache.get_relay() do
       conn
-      |> put_resp_header("content-type", "application/activity+json")
+      |> put_resp_content_type("application/activity+json")
       |> json(ActorView.render("actor.json", %{actor: actor}))
     end
   end
@@ -194,7 +195,7 @@ defmodule Mobilizon.Web.ActivityPubController do
          page <- max(page, 1),
          %Actor{} = actor <- Actors.get_local_actor_by_name_with_preload(name) do
       conn
-      |> put_resp_header("content-type", "application/activity+json")
+      |> put_resp_content_type("application/activity+json")
       |> json(
         ActorView.render("#{collection}.json", %{
           actor: actor,
@@ -208,7 +209,7 @@ defmodule Mobilizon.Web.ActivityPubController do
   defp actor_collection(conn, collection, %{"name" => name}) do
     with %Actor{} = actor <- Actors.get_local_actor_by_name_with_preload(name) do
       conn
-      |> put_resp_header("content-type", "application/activity+json")
+      |> put_resp_content_type("application/activity+json")
       |> json(
         ActorView.render("#{collection}.json", %{
           actor: actor,
diff --git a/lib/web/controllers/page_controller.ex b/lib/web/controllers/page_controller.ex
index cc00560a5..9afb1f779 100644
--- a/lib/web/controllers/page_controller.ex
+++ b/lib/web/controllers/page_controller.ex
@@ -141,11 +141,13 @@ defmodule Mobilizon.Web.PageController do
           %Tombstone{} ->
             conn
             |> put_status(:gone)
+            |> maybe_add_content_type_header()
             |> render(object_type, object: object)
 
           _ ->
             conn
             |> maybe_add_noindex_header(object)
+            |> maybe_add_content_type_header()
             |> render(object_type, object: object)
         end
 
@@ -199,4 +201,14 @@ defmodule Mobilizon.Web.PageController do
   @spec is_person?(Actor.t()) :: boolean()
   defp is_person?(%Actor{type: :Person}), do: true
   defp is_person?(_), do: false
+
+  defp maybe_add_content_type_header(conn) do
+    case get_format(conn) do
+      "html" ->
+        conn
+
+      "activity-json" ->
+        put_resp_content_type(conn, "application/activity+json")
+    end
+  end
 end

From 8e7907ec9eacdc48238afe5de2616461eb31286c Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Tue, 22 Nov 2022 12:02:55 +0100
Subject: [PATCH 6/9] Don't notify group members & followers from new draft
 event

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 lib/service/workers/event_delayed_notification_worker.ex | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/lib/service/workers/event_delayed_notification_worker.ex b/lib/service/workers/event_delayed_notification_worker.ex
index a58a91af2..5eaa07092 100644
--- a/lib/service/workers/event_delayed_notification_worker.ex
+++ b/lib/service/workers/event_delayed_notification_worker.ex
@@ -14,11 +14,11 @@ defmodule Mobilizon.Service.Workers.EventDelayedNotificationWorker do
   @impl Oban.Worker
   def perform(%Job{args: %{"action" => "notify_of_new_event", "event_uuid" => event_uuid}}) do
     case Events.get_event_by_uuid_with_preload(event_uuid) do
-      %Event{} = event ->
+      %Event{draft: false} = event ->
         Group.notify_of_new_event(event)
 
       nil ->
-        # Event deleted inbetween, no worries, just ignore
+        # Event still a draft or event deleted inbetween, no worries, just ignore
         :ok
     end
   end
@@ -44,7 +44,7 @@ defmodule Mobilizon.Service.Workers.EventDelayedNotificationWorker do
         )
 
       _ ->
-        # Event deleted inbetween, no worries, just ignore
+        # Event still a draft or event deleted inbetween, no worries, just ignore
         :ok
     end
   end

From 4073eab201bf2e7f08927e60fe5f3ebad8a47e21 Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Tue, 22 Nov 2022 12:17:52 +0100
Subject: [PATCH 7/9] Fix approving/rejecting group members

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 js/src/views/Group/GroupMembers.vue | 39 +++++++++++++++++++++++++----
 1 file changed, 34 insertions(+), 5 deletions(-)

diff --git a/js/src/views/Group/GroupMembers.vue b/js/src/views/Group/GroupMembers.vue
index 7308ede27..ce7018365 100644
--- a/js/src/views/Group/GroupMembers.vue
+++ b/js/src/views/Group/GroupMembers.vue
@@ -184,7 +184,7 @@
             <o-button
               variant="success"
               v-if="props.row.role === MemberRole.NOT_APPROVED"
-              @click="approveMember(props.row.id)"
+              @click="approveMember({ memberId: props.row.id })"
               icon-left="check"
               >{{ t("Approve member") }}</o-button
             >
@@ -238,7 +238,6 @@
 </template>
 
 <script lang="ts" setup>
-import { FETCH_GROUP } from "@/graphql/group";
 import { MemberRole } from "@/types/enums";
 import { IMember } from "@/types/actor/member.model";
 import RouteName from "@/router/name";
@@ -312,6 +311,12 @@ const {
   refetchQueries: [
     {
       query: GROUP_MEMBERS,
+      variables: {
+        groupName: props.preferredUsername,
+        page: page.value,
+        limit: MEMBERS_PER_PAGE,
+        roles: roles.value,
+      },
     },
   ],
 }));
@@ -363,6 +368,12 @@ const {
   refetchQueries: [
     {
       query: GROUP_MEMBERS,
+      variables: {
+        groupName: props.preferredUsername,
+        page: page.value,
+        limit: MEMBERS_PER_PAGE,
+        roles: roles.value,
+      },
     },
   ],
 }));
@@ -423,7 +434,20 @@ const {
   onDone: onApproveMemberDone,
   onError: onApproveMemberError,
 } = useMutation<{ approveMember: IMember }, { memberId: string }>(
-  APPROVE_MEMBER
+  APPROVE_MEMBER,
+  {
+    refetchQueries: [
+      {
+        query: GROUP_MEMBERS,
+        variables: {
+          groupName: props.preferredUsername,
+          page: page.value,
+          limit: MEMBERS_PER_PAGE,
+          roles: roles.value,
+        },
+      },
+    ],
+  }
 );
 
 onApproveMemberDone(() => {
@@ -452,8 +476,13 @@ const {
 >(UPDATE_MEMBER, () => ({
   refetchQueries: [
     {
-      query: FETCH_GROUP,
-      variables: { name: props.preferredUsername },
+      query: GROUP_MEMBERS,
+      variables: {
+        groupName: props.preferredUsername,
+        page: page.value,
+        limit: MEMBERS_PER_PAGE,
+        roles: roles.value,
+      },
     },
   ],
 }));

From fde0c9b5d817c133eeb1135ac09977704a2b6316 Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Tue, 22 Nov 2022 12:21:26 +0100
Subject: [PATCH 8/9] Register missing icon

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 js/src/components/core/MaterialIcon.vue | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/js/src/components/core/MaterialIcon.vue b/js/src/components/core/MaterialIcon.vue
index 59bdd97a3..42541c7cb 100644
--- a/js/src/components/core/MaterialIcon.vue
+++ b/js/src/components/core/MaterialIcon.vue
@@ -246,6 +246,8 @@ const icons: Record<string, () => Promise<any>> = {
     import(
       `../../../node_modules/vue-material-design-icons/BellOffOutline.vue`
     ),
+  ExitToApp: () =>
+    import(`../../../node_modules/vue-material-design-icons/ExitToApp.vue`),
 };
 
 const props = withDefaults(

From 9eab9f1bbb62303a00b3ea4bb482924191d005f5 Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Tue, 22 Nov 2022 12:47:38 +0100
Subject: [PATCH 9/9] Fix comment display

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 js/src/components/Comment/EventComment.vue |  7 ++++---
 js/tailwind.config.js                      | 20 ++++++++++----------
 2 files changed, 14 insertions(+), 13 deletions(-)

diff --git a/js/src/components/Comment/EventComment.vue b/js/src/components/Comment/EventComment.vue
index 48e7fcf36..90e1e01af 100644
--- a/js/src/components/Comment/EventComment.vue
+++ b/js/src/components/Comment/EventComment.vue
@@ -1,12 +1,12 @@
 <template>
   <li
+    class="bg-white dark:bg-zinc-800 rounded p-2"
     :class="{
       reply: comment.inReplyToComment,
-      'bg-purple-2': comment.isAnnouncement,
-      'bg-violet-1': commentSelected,
+      'bg-mbz-purple-50 dark:bg-mbz-purple-500': comment.isAnnouncement,
+      'bg-mbz-bluegreen-50 dark:bg-mbz-bluegreen-600': commentSelected,
       'shadow-none': !rootComment,
     }"
-    class="bg-white dark:bg-zinc-900 rounded p-2"
   >
     <article :id="commentId" dir="auto" class="mbz-comment">
       <div>
@@ -53,6 +53,7 @@
           v-html="comment.text"
           dir="auto"
           :lang="comment.language"
+          class="prose dark:prose-invert xl:prose-lg !max-w-full" :class="{ 'text-black dark:text-white': comment.isAnnouncement}"
         />
         <div v-else>{{ t("[This comment has been deleted]") }}</div>
         <nav class="flex gap-1 mt-1" v-if="!comment.deletedAt">
diff --git a/js/tailwind.config.js b/js/tailwind.config.js
index a5486facb..3c21e8e5b 100644
--- a/js/tailwind.config.js
+++ b/js/tailwind.config.js
@@ -38,16 +38,16 @@ module.exports = {
         },
         "mbz-purple": {
           DEFAULT: "#424056",
-          50: "#9C9AB4",
-          100: "#918EAB",
-          200: "#7A779A",
-          300: "#666385",
-          400: "#54516D",
-          500: "#424056",
-          600: "#292836",
-          700: "#111016",
-          800: "#000000",
-          900: "#000000",
+          50: "#CAC9D7",
+          100: "#BEBDCE",
+          200: "#A8A6BC",
+          300: "#918EAB",
+          400: "#7A779A",
+          500: "#666385",
+          600: "#54516D",
+          700: "#424056",
+          800: "#292836",
+          900: "#111016",
         },
         "mbz-bluegreen": {
           DEFAULT: "#1E7D97",