From 76ff11ceae7c73998f04bc2b2f9b5bcd5896c68e Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Wed, 12 Oct 2022 18:27:05 +0200
Subject: [PATCH 1/6] Don't add empty search parameters to global search engine

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 lib/service/global_search/search_mobilizon.ex | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/service/global_search/search_mobilizon.ex b/lib/service/global_search/search_mobilizon.ex
index 238a722d3..4347263e4 100644
--- a/lib/service/global_search/search_mobilizon.ex
+++ b/lib/service/global_search/search_mobilizon.ex
@@ -70,7 +70,7 @@ defmodule Mobilizon.Service.GlobalSearch.SearchMobilizon do
         :count,
         :sortBy
       ])
-      |> Keyword.reject(fn {_key, val} -> is_nil(val) end)
+      |> Keyword.reject(fn {_key, val} -> is_nil(val) or val == "" end)
 
     events_url = "#{search_endpoint()}#{@search_events_api}?#{encode(options)}"
     Logger.debug("Calling global search engine url #{events_url}")
@@ -117,7 +117,7 @@ defmodule Mobilizon.Service.GlobalSearch.SearchMobilizon do
         :bbox,
         :sortBy
       ])
-      |> Keyword.reject(fn {_key, val} -> is_nil(val) end)
+      |> Keyword.reject(fn {_key, val} -> is_nil(val) or val == "" end)
 
     groups_url = "#{search_endpoint()}#{@search_groups_api}?#{encode(options)}"
     Logger.debug("Calling global search engine url #{groups_url}")

From c051e26380cf9e5ba27d05d5b671be30c394e6e2 Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Wed, 12 Oct 2022 18:27:26 +0200
Subject: [PATCH 2/6] Fix getting categories from global search engine

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

diff --git a/lib/service/global_search/search_mobilizon.ex b/lib/service/global_search/search_mobilizon.ex
index 4347263e4..91c47f0e8 100644
--- a/lib/service/global_search/search_mobilizon.ex
+++ b/lib/service/global_search/search_mobilizon.ex
@@ -187,7 +187,7 @@ defmodule Mobilizon.Service.GlobalSearch.SearchMobilizon do
       ends_on: parse_date(data["endTime"]),
       url: data["url"],
       picture: picture,
-      category: String.to_existing_atom(Categories.get_category(data["category"])),
+      category: data["category"] |> Categories.get_category() |> String.downcase() |> String.to_existing_atom(),
       organizer_actor: %Actor{
         id: data["creator"]["id"],
         name: data["creator"]["displayName"],

From fca3720bdbd332448544c604720ec0839b176c60 Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Wed, 12 Oct 2022 19:04:33 +0200
Subject: [PATCH 3/6] Remove unused deps

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 js/package.json                           |  4 ----
 js/src/components/Comment/CommentTree.vue |  1 -
 js/src/components/Post/PostListItem.vue   |  1 -
 js/yarn.lock                              | 22 ----------------------
 4 files changed, 28 deletions(-)

diff --git a/js/package.json b/js/package.json
index b15ba24bf..067991cac 100644
--- a/js/package.json
+++ b/js/package.json
@@ -20,7 +20,6 @@
     "@absinthe/socket": "^0.2.1",
     "@absinthe/socket-apollo-link": "^0.2.1",
     "@apollo/client": "^3.3.16",
-    "@headlessui/vue": "^1.6.7",
     "@oruga-ui/oruga-next": "^0.5.5",
     "@sentry/tracing": "^7.1",
     "@sentry/vue": "^7.1",
@@ -58,8 +57,6 @@
     "apollo-absinthe-upload-link": "^1.5.0",
     "autoprefixer": "^10",
     "blurhash": "^2.0.0",
-    "bulma": "^0.9.4",
-    "bulma-divider": "^0.2.0",
     "date-fns": "^2.16.0",
     "date-fns-tz": "^1.1.6",
     "floating-vue": "^2.0.0-beta.17",
@@ -85,7 +82,6 @@
     "vue-i18n": "9",
     "vue-material-design-icons": "^5.1.2",
     "vue-matomo": "^4.1.0",
-    "vue-meta": "^2.3.1",
     "vue-plausible": "^1.3.1",
     "vue-router": "4",
     "vue-scrollto": "^2.17.1",
diff --git a/js/src/components/Comment/CommentTree.vue b/js/src/components/Comment/CommentTree.vue
index 38ba89e24..22d73c1ba 100644
--- a/js/src/components/Comment/CommentTree.vue
+++ b/js/src/components/Comment/CommentTree.vue
@@ -377,7 +377,6 @@ const isConnected = computed((): boolean => {
 
 <style lang="scss" scoped>
 // @use "@/styles/_mixins" as *;
-// // @import "node_modules/bulma/sass/utilities/mixins.sass";
 // form.new-comment {
 //   padding-bottom: 1rem;
 
diff --git a/js/src/components/Post/PostListItem.vue b/js/src/components/Post/PostListItem.vue
index 2a2ef85f7..4ec0f7239 100644
--- a/js/src/components/Post/PostListItem.vue
+++ b/js/src/components/Post/PostListItem.vue
@@ -85,7 +85,6 @@ const isBeforeLastWeek = computed((): boolean => {
 </script>
 <style lang="scss" scoped>
 @use "@/styles/_mixins" as *;
-// @import "node_modules/bulma/sass/utilities/mixins.sass";
 
 // .post-minimalist-card-wrapper {
 //   display: grid;
diff --git a/js/yarn.lock b/js/yarn.lock
index 308d70b04..1de109779 100644
--- a/js/yarn.lock
+++ b/js/yarn.lock
@@ -1073,11 +1073,6 @@
   resolved "https://registry.yarnpkg.com/@graphql-typed-document-node/core/-/core-3.1.1.tgz#076d78ce99822258cf813ecc1e7fa460fa74d052"
   integrity sha512-NQ17ii0rK1b34VZonlmT2QMJFI70m0TRwbknO/ihlbatXyaktDhN/98vBiUU6kNBPljqGqyIrl2T4nY2RpFANg==
 
-"@headlessui/vue@^1.6.7":
-  version "1.7.3"
-  resolved "https://registry.yarnpkg.com/@headlessui/vue/-/vue-1.7.3.tgz#116d13391552436ea1e9e88c69f0e9d7d2585c45"
-  integrity sha512-Is4iakKts9u9E0+jEZNzoJpBjwq2SamwEIoEl2RlyYSu6Zco536GsPXaQEfg/o7Eyc1GUUlcL+dJd4Rt7qyf7A==
-
 "@histoire/app@^0.11.3":
   version "0.11.3"
   resolved "https://registry.yarnpkg.com/@histoire/app/-/app-0.11.3.tgz#6b353a4b60a61eef4170c6c91e8e997d69caf0bc"
@@ -2546,16 +2541,6 @@ builtin-modules@^3.1.0:
   resolved "https://registry.yarnpkg.com/builtin-modules/-/builtin-modules-3.3.0.tgz#cae62812b89801e9656336e46223e030386be7b6"
   integrity sha512-zhaCDicdLuWN5UbN5IMnFqNMhNfo919sH85y2/ea+5Yg9TsTkeZxpL+JLbp6cgYFS4sRLp3YV4S6yDuqVWHYOw==
 
-bulma-divider@^0.2.0:
-  version "0.2.0"
-  resolved "https://registry.yarnpkg.com/bulma-divider/-/bulma-divider-0.2.0.tgz#a9b4d9fe8b270c7cb7573023c575062bc62616f3"
-  integrity sha512-REe3k56GECRfDaqFjC8cwLhV4RxXmV0RubuzDJqwior9wlJcdHlN0qfW0tvUX+qphikaTQegIeRuhjRIAqkjkw==
-
-bulma@^0.9.4:
-  version "0.9.4"
-  resolved "https://registry.yarnpkg.com/bulma/-/bulma-0.9.4.tgz#0ca8aeb1847a34264768dba26a064c8be72674a1"
-  integrity sha512-86FlT5+1GrsgKbPLRRY7cGDg8fsJiP/jzTqXXVqiUZZ2aZT8uemEOHlU1CDU+TxklPEZ11HZNNWclRBBecP4CQ==
-
 c8@^7.12.0:
   version "7.12.0"
   resolved "https://registry.yarnpkg.com/c8/-/c8-7.12.0.tgz#402db1c1af4af5249153535d1c84ad70c5c96b14"
@@ -6404,13 +6389,6 @@ vue-matomo@^4.1.0:
   resolved "https://registry.yarnpkg.com/vue-matomo/-/vue-matomo-4.2.0.tgz#d65e369e4ead1d95ef790bef3627512cac3d25e9"
   integrity sha512-m5hCw7LH3wPDcERaF4sp/ojR9sEx7Rl8TpOyH/4jjQxMF2DuY/q5pO+i9o5Dx+BXLSa9+IQ0qhAbWYRyESQXmA==
 
-vue-meta@^2.3.1:
-  version "2.4.0"
-  resolved "https://registry.yarnpkg.com/vue-meta/-/vue-meta-2.4.0.tgz#a419fb4b4135ce965dab32ec641d1989c2ee4845"
-  integrity sha512-XEeZUmlVeODclAjCNpWDnjgw+t3WA6gdzs6ENoIAgwO1J1d5p1tezDhtteLUFwcaQaTtayRrsx7GL6oXp/m2Jw==
-  dependencies:
-    deepmerge "^4.2.2"
-
 vue-plausible@^1.3.1:
   version "1.3.2"
   resolved "https://registry.yarnpkg.com/vue-plausible/-/vue-plausible-1.3.2.tgz#c797ee03a7c8849dc847e706c5250aa10908dd0f"

From e4ba0f21eee3ece89238952762166f39c0d9e588 Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Wed, 12 Oct 2022 19:10:39 +0200
Subject: [PATCH 4/6] Only show one pagination bar when searching in both
 events & groups

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 js/src/views/SearchView.vue | 100 +++++++++++++++++++-----------------
 1 file changed, 53 insertions(+), 47 deletions(-)

diff --git a/js/src/views/SearchView.vue b/js/src/views/SearchView.vue
index 9ee3600ae..df1cc1c59 100644
--- a/js/src/views/SearchView.vue
+++ b/js/src/views/SearchView.vue
@@ -503,54 +503,45 @@
           <o-notification v-if="features && !features.groups" variant="danger">
             {{ t("Groups are not enabled on this instance.") }}
           </o-notification>
-          <div v-else-if="searchGroups && searchGroups?.total > 0">
-            <GroupCard
-              v-for="group in searchGroups?.elements"
-              :group="group"
-              :key="group.id"
-              :isRemoteGroup="group.__typename === 'GroupResult'"
-              :isLoggedIn="currentUser?.isLoggedIn"
-              mode="row"
-            />
-            <o-pagination
-              v-if="searchGroups && searchGroups?.total > GROUP_PAGE_LIMIT"
-              :total="searchGroups?.total"
-              v-model:current="groupPage"
-              :per-page="GROUP_PAGE_LIMIT"
-              :aria-next-label="t('Next page')"
-              :aria-previous-label="t('Previous page')"
-              :aria-page-label="t('Page')"
-              :aria-current-label="t('Current page')"
-            >
-            </o-pagination>
-          </div>
+          <GroupCard
+            v-else-if="searchGroups && searchGroups?.total > 0"
+            v-for="group in searchGroups?.elements"
+            :group="group"
+            :key="group.id"
+            :isRemoteGroup="group.__typename === 'GroupResult'"
+            :isLoggedIn="currentUser?.isLoggedIn"
+            mode="row"
+          />
           <o-notification v-else-if="searchLoading === false" variant="danger">
             {{ t("No groups found") }}
           </o-notification>
-          <div v-if="searchEvents && searchEvents.total > 0">
-            <event-card
-              mode="row"
-              v-for="event in searchEvents?.elements"
-              :event="event"
-              :key="event.uuid"
-              :options="{
-                isRemoteEvent: event.__typename === 'EventResult',
-                isLoggedIn: currentUser?.isLoggedIn,
-              }"
-              class="my-4"
-            />
-            <o-pagination
-              v-if="searchEvents && searchEvents?.total > EVENT_PAGE_LIMIT"
-              :total="searchEvents.total"
-              v-model:current="eventPage"
-              :per-page="EVENT_PAGE_LIMIT"
-              :aria-next-label="t('Next page')"
-              :aria-previous-label="t('Previous page')"
-              :aria-page-label="t('Page')"
-              :aria-current-label="t('Current page')"
-            >
-            </o-pagination>
-          </div>
+          <event-card
+            v-if="searchEvents && searchEvents.total > 0"
+            mode="row"
+            v-for="event in searchEvents?.elements"
+            :event="event"
+            :key="event.uuid"
+            :options="{
+              isRemoteEvent: event.__typename === 'EventResult',
+              isLoggedIn: currentUser?.isLoggedIn,
+            }"
+            class="my-4"
+          />
+          <o-pagination
+            v-if="
+              (searchEvents && searchEvents?.total > EVENT_PAGE_LIMIT) ||
+              (searchGroups && searchGroups?.total > GROUP_PAGE_LIMIT)
+            "
+            :total="
+              Math.max(searchEvents?.total ?? 0, searchGroups?.total ?? 0)
+            "
+            v-model:current="page"
+            :per-page="EVENT_PAGE_LIMIT"
+            :aria-next-label="t('Next page')"
+            :aria-previous-label="t('Previous page')"
+            :aria-page-label="t('Page')"
+            :aria-current-label="t('Current page')"
+          />
           <o-notification v-else-if="searchLoading === false" variant="info">
             <p>{{ t("No events found") }}</p>
             <p v-if="searchIsUrl && !currentUser?.id">
@@ -770,6 +761,7 @@ const arrayTransformer: RouteQueryTransformer<string[]> = {
   },
 };
 
+const page = useRouteQuery("page", 1, integerTransformer);
 const eventPage = useRouteQuery("eventPage", 1, integerTransformer);
 const groupPage = useRouteQuery("groupPage", 1, integerTransformer);
 
@@ -783,6 +775,18 @@ const contentType = useRouteQuery(
   ContentType.ALL,
   enumTransformer(ContentType)
 );
+
+watch(contentType, (newContentType: ContentType) => {
+  switch (newContentType) {
+    case ContentType.ALL:
+      page.value = 1;
+    case ContentType.EVENTS:
+      eventPage.value = 1;
+    case ContentType.GROUPS:
+      groupPage.value = 1;
+  }
+});
+
 const isOnline = useRouteQuery("isOnline", false, booleanTransformer);
 const categoryOneOf = useRouteQuery("categoryOneOf", [], arrayTransformer);
 const statusOneOf = useRouteQuery(
@@ -1168,8 +1172,10 @@ const { result: searchElementsResult, loading: searchLoading } = useQuery<{
   beginsOn: start.value,
   endsOn: end.value,
   radius: geoHashLocation.value ? radius.value : undefined,
-  eventPage: eventPage.value,
-  groupPage: groupPage.value,
+  eventPage:
+    contentType.value === ContentType.ALL ? page.value : eventPage.value,
+  groupPage:
+    contentType.value === ContentType.ALL ? page.value : groupPage.value,
   limit: EVENT_PAGE_LIMIT,
   type: isOnline.value ? "ONLINE" : undefined,
   categoryOneOf: categoryOneOf.value,

From fb0238d1b7f1c1fdd3e08239ef9ffddb978ff9d2 Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Wed, 12 Oct 2022 18:40:54 +0200
Subject: [PATCH 5/6] Fix: Run build multiarch release on tags too

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 .gitlab-ci.yml | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 2a68f222e..e800dcdb4 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -342,11 +342,9 @@ multi-arch-release:
       - ARCH: ["arm", "arm64"]
   rules:
     # arm64 is allowed to fail
-    - if: '$CI_PROJECT_NAMESPACE == "framasoft" && $CI_PIPELINE_SOURCE == "schedule" && $ARCH == "arm64"'
-      when: on_success
+    - if: '$CI_PROJECT_NAMESPACE == "framasoft" && ($CI_PIPELINE_SOURCE == "schedule" || $CI_COMMIT_TAG != null) && $ARCH == "arm64"'
       allow_failure: true
-    - if: '$CI_PROJECT_NAMESPACE == "framasoft" && $CI_PIPELINE_SOURCE == "schedule" && $ARCH'
-      when: on_success
+    - if: '$CI_PROJECT_NAMESPACE == "framasoft" && ($CI_PIPELINE_SOURCE == "schedule" || $CI_COMMIT_TAG != null) && $ARCH'
       allow_failure: false
   timeout: 3h
   allow_failure: true

From 699465199eec48753b60bd9749d7132eacf122ef Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Wed, 12 Oct 2022 19:30:34 +0200
Subject: [PATCH 6/6] Lint fixes

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 js/src/views/Event/EventView.vue              |  7 +--
 js/src/views/SearchView.vue                   | 47 ++++++++++---------
 lib/service/global_search/search_mobilizon.ex |  6 ++-
 3 files changed, 32 insertions(+), 28 deletions(-)

diff --git a/js/src/views/Event/EventView.vue b/js/src/views/Event/EventView.vue
index 3ff5ad6e5..dbeb09900 100755
--- a/js/src/views/Event/EventView.vue
+++ b/js/src/views/Event/EventView.vue
@@ -188,12 +188,7 @@ import {
   EVENT_PERSON_PARTICIPATION,
   // EVENT_PERSON_PARTICIPATION_SUBSCRIPTION_CHANGED,
 } from "@/graphql/event";
-import {
-  displayName,
-  IActor,
-  IPerson,
-  usernameWithDomain,
-} from "@/types/actor";
+import { displayName, IPerson, usernameWithDomain } from "@/types/actor";
 import DateCalendarIcon from "@/components/Event/DateCalendarIcon.vue";
 import MultiCard from "@/components/Event/MultiCard.vue";
 import RouteName from "@/router/name";
diff --git a/js/src/views/SearchView.vue b/js/src/views/SearchView.vue
index df1cc1c59..a37ca71ce 100644
--- a/js/src/views/SearchView.vue
+++ b/js/src/views/SearchView.vue
@@ -503,30 +503,32 @@
           <o-notification v-if="features && !features.groups" variant="danger">
             {{ t("Groups are not enabled on this instance.") }}
           </o-notification>
-          <GroupCard
-            v-else-if="searchGroups && searchGroups?.total > 0"
-            v-for="group in searchGroups?.elements"
-            :group="group"
-            :key="group.id"
-            :isRemoteGroup="group.__typename === 'GroupResult'"
-            :isLoggedIn="currentUser?.isLoggedIn"
-            mode="row"
-          />
+          <div v-else-if="searchGroups && searchGroups?.total > 0">
+            <GroupCard
+              v-for="group in searchGroups?.elements"
+              :group="group"
+              :key="group.id"
+              :isRemoteGroup="group.__typename === 'GroupResult'"
+              :isLoggedIn="currentUser?.isLoggedIn"
+              mode="row"
+            />
+          </div>
           <o-notification v-else-if="searchLoading === false" variant="danger">
             {{ t("No groups found") }}
           </o-notification>
-          <event-card
-            v-if="searchEvents && searchEvents.total > 0"
-            mode="row"
-            v-for="event in searchEvents?.elements"
-            :event="event"
-            :key="event.uuid"
-            :options="{
-              isRemoteEvent: event.__typename === 'EventResult',
-              isLoggedIn: currentUser?.isLoggedIn,
-            }"
-            class="my-4"
-          />
+          <div v-if="searchEvents && searchEvents.total > 0">
+            <event-card
+              mode="row"
+              v-for="event in searchEvents?.elements"
+              :event="event"
+              :key="event.uuid"
+              :options="{
+                isRemoteEvent: event.__typename === 'EventResult',
+                isLoggedIn: currentUser?.isLoggedIn,
+              }"
+              class="my-4"
+            />
+          </div>
           <o-pagination
             v-if="
               (searchEvents && searchEvents?.total > EVENT_PAGE_LIMIT) ||
@@ -780,10 +782,13 @@ watch(contentType, (newContentType: ContentType) => {
   switch (newContentType) {
     case ContentType.ALL:
       page.value = 1;
+      break;
     case ContentType.EVENTS:
       eventPage.value = 1;
+      break;
     case ContentType.GROUPS:
       groupPage.value = 1;
+      break;
   }
 });
 
diff --git a/lib/service/global_search/search_mobilizon.ex b/lib/service/global_search/search_mobilizon.ex
index 91c47f0e8..8850f790c 100644
--- a/lib/service/global_search/search_mobilizon.ex
+++ b/lib/service/global_search/search_mobilizon.ex
@@ -187,7 +187,11 @@ defmodule Mobilizon.Service.GlobalSearch.SearchMobilizon do
       ends_on: parse_date(data["endTime"]),
       url: data["url"],
       picture: picture,
-      category: data["category"] |> Categories.get_category() |> String.downcase() |> String.to_existing_atom(),
+      category:
+        data["category"]
+        |> Categories.get_category()
+        |> String.downcase()
+        |> String.to_existing_atom(),
       organizer_actor: %Actor{
         id: data["creator"]["id"],
         name: data["creator"]["displayName"],