refactor(toolbar): delete the Actions section to have only the toolbar (#868)

* fix(toolbar): repairs the behaviour of the modal accept & refusal button

* refactor(toolbar): Delete the Actions section to have only the toolbar

* refactor(toolbar): Delete space to have less scrolling & delete inexistent "last-message" class

* test(snapshot): update snapshots

* test(e2e): update cumcumber tests

* fix(moderation): fix error 400 when rejecting member

Author: rebeccadumazert

e2e/features/moderations/jean_bon_do_not_join_dinum.feature

@@ -21,40 +21,26 @@ Fonctionnalité: Moderation blockante à refuser
     Quand sur la même ligne je clique sur "➡️"
     Alors je vois "Jean Bon veut rejoindre l'organisation « DINUM » avec l’adresse jeanbon@yopmail.com"
 
-
-  Scénario: Jean Bon ne peux pas rejoindre l'organisation DINUM
-    Quand je clique sur "Je refuse ce membre ❌"
-    Alors je vois "Motif de refus :"
-
-    Soit je vais à l'intérieur de la section nommé "la section motif de refus"
-    Quand je saisie le mot "Nom de domaine introuvable{enter}" dans la boîte à texte nommée "Recherche d'une réponse type"
-    # Alors je dois voir une boîte à texte nommée "Message" et contenant:
-    #   """
-    #   Nous n'avons pas été en mesure de vérifier le nom de domaine de votre adresse mail
-    #   """
-
-    Quand je clique sur "Notifier le membre et terminer"
-    * je reinitialise le contexte
-    Alors je vois "Liste des moderations"
-    Alors je ne vois pas "13002526500013"
-
-    Quand je reviens en avant
-    Alors je vois "Cette modération a été marqué comme traitée"
-
-    Quand je clique sur "Commentaires"
-    Alors je vois "Rejeté par user@yopmail.com"
-    Alors je vois 'Raison : "Nom de domaine introuvable"'
-
   Scénario: Le modérateur le refuse avec la barre d'outils
     Quand je clique sur "❌ Refuser"
     Alors je vois "Vous refusez la demande de jeanbon@yopmail.com"
 
     Soit je vais à l'intérieur du dialogue nommé "la modale de refus"
     Quand je saisie le mot "Nom de domaine introuvable{enter}" dans la boîte à texte nommée "Recherche d'une réponse type"
     * je clique sur "Notifier le membre et terminer"
+    Etant donné que je reinitialise le contexte
+    Quand je reviens en avant
     Alors je vois "Cette modération a été marqué comme traitée le"
     * je vois "Rejeté par user@yopmail.com"
     * je vois 'Raison : "Nom de domaine introuvable"'
 
     Alors je vois "Liste des moderations"
     Alors je ne vois pas "13002526500013"
+
+    Quand je reviens en avant
+    Alors je vois "Cette modération a été marqué comme traitée"
+
+
+    Quand je clique sur "Commentaires"
+    Alors je vois "Rejeté par user@yopmail.com"
+    Alors je vois 'Raison : "Nom de domaine introuvable"'

e2e/features/moderations/jean_bon_join_abracadabra.feature

@@ -48,15 +48,6 @@ Fonctionnalité: Moderation blockante à accepter
     * je clique sur le bouton "❎ Domaine externe"
     Alors sur la même ligne je vois "❎"
 
-  Scénario: Envoyer l'email « Votre demande a été traitée »
-    Quand je clique sur "Je valide ce membre ✅"
-    Et je clique sur "Terminer"
-    * je vois "Cette modération a été marqué comme traitée le"
-    * je vois "Validé par user@yopmail.com"
-
-    Alors je vois "Liste des moderations"
-    * je ne vois pas "51935970700022"
-
   Scénario: Le modérateur le valide avec la barre d'outils
     Quand je clique sur "✅ Accepter"
     Alors je vois "A propos de jeanbon@yopmail.com, je valide :"

e2e/features/moderations/marie_bon_join_bosch.feature

@@ -21,9 +21,9 @@ Fonctionnalité: Moderation non blockante
   Scénario: Le nom de domaine est vérifié
     Soit le tableau sous le title "domaine connu dans l'organisation"
     * le tableau est vide
-    Quand je clique sur "Je valide ce membre ✅"
+    Quand je clique sur "✅ Accepter"
     Quand je clique sur "J’autorise le domaine fr.bosch.com pour toute l’organisation"
-    Quand je clique sur "Terminer"
+    Quand je clique sur "Notifier le membre et terminer"
 
     Alors je vois "Liste des moderations"
     Quand je clique sur "Voir les demandes traitées"
@@ -37,9 +37,9 @@ Fonctionnalité: Moderation non blockante
   Scénario: Marie est un membre interne de l'organization.
     Soit le tableau sous le title "0 membre connu dans l’organisation"
     * le tableau est vide
-    Quand je clique sur "Je valide ce membre ✅"
+    Quand je clique sur "✅ Accepter"
     Et je clique sur "Ajouter Marie à l'organisation EN TANT QU'INTERNE"
-    Quand je clique sur "Terminer"
+    Quand je clique sur "Notifier le membre et terminer"
 
     Alors je vois "Liste des moderations"
     Quand je clique sur "Voir les demandes traitées"
@@ -54,9 +54,9 @@ Fonctionnalité: Moderation non blockante
     Soit le tableau sous le title "0 membre connu dans l’organisation"
     * le tableau est vide
 
-    Quand je clique sur "Je valide ce membre ✅"
+    Quand je clique sur "✅ Accepter"
     * je clique sur "Ajouter Marie à l'organisation EN TANT QU'EXTERNE"
-    * je clique sur "Terminer"
+    * je clique sur "Notifier le membre et terminer"
 
     Alors je vois "Liste des moderations"
     Quand je clique sur "Voir les demandes traitées"

packages/~/moderations/api/src/:id/email/page/Zammad.tsx

@@ -43,12 +43,8 @@ function List() {
   return (
     <ul class="list-none">
       <ShowMore />
-      {articles.map((article, index) => (
-        <li
-          class={index === articles.length - 1 ? "last-message" : ""}
-          id={`${article.id}`}
-          key={`${article.id}`}
-        >
+      {articles.map((article) => (
+        <li id={`${article.id}`} key={`${article.id}`}>
           <p class="text-center text-sm font-bold">
             <LocalTime date={article.created_at} />
           </p>

packages/~/moderations/ui/src/Actions/AcceptModal.tsx

@@ -48,6 +48,7 @@ export async function AcceptModal({
         hx-swap="none"
         _={`
             on submit
+              add .hidden to #acceptModal
               wait for ${Htmx_Events.enum.afterOnLoad}
               go to the top of body smoothly
               wait 2s
@@ -67,14 +68,7 @@ export async function AcceptModal({
           <TagInput />
         </div>
         <div>
-          <button
-            _={`
-              on click
-                add .hidden to #acceptModal
-            `}
-            class={`${button()} w-full justify-center`}
-            type="submit"
-          >
+          <button class={`${button()} w-full justify-center`} type="submit">
             Notifier le membre et terminer
           </button>
         </div>

packages/~/moderations/ui/src/Actions/Actions.tsx

@@ -2,14 +2,8 @@
 
 import { hyper_ref } from "@~/app.core/html";
 import { z_email_domain } from "@~/app.core/schema/z_email_domain";
-import { button } from "@~/app.ui/button";
-import { hx_urls } from "@~/app.urls";
-import { MessageInfo } from "@~/moderations.ui/MessageInfo";
 import { AcceptModal } from "./AcceptModal";
 import { context, type Values } from "./context";
-import { Desicison } from "./Desicison";
-import { MemberInvalid } from "./MemberInvalid";
-import { MemberValid } from "./MemberValid";
 import { RefusalModal } from "./RefusalModal";
 import { Toolbar } from "./Toolbar";
 
@@ -24,12 +18,6 @@ export async function Actions({ value }: ActionProps) {
 
   const { user } = moderation;
 
-  const hx_moderation_reprocess_props = await hx_urls.moderations[
-    ":id"
-  ].$procedures.reprocess.$patch({
-    param: { id: moderation.id.toString() },
-  });
-
   const domain = z_email_domain.parse(moderation.user.email, {
     path: ["user.email"],
   });
@@ -44,28 +32,6 @@ export async function Actions({ value }: ActionProps) {
         ...value,
       }}
     >
-      <div class="bg-[var(--background-alt-blue-france)] p-8">
-        <h2>Actions de modération</h2>
-
-        <MessageInfo moderation={moderation} />
-
-        <hr class="bg-none" />
-        {moderation.moderated_at ? (
-          <button
-            class={button({ size: "sm", type: "tertiary" })}
-            {...hx_moderation_reprocess_props}
-            hx-swap="none"
-          >
-            Retraiter
-          </button>
-        ) : (
-          <>
-            <Desicison />
-            <MemberValid />
-            <MemberInvalid />
-          </>
-        )}
-      </div>
       <Toolbar />
       <AcceptModal userEmail={user.email} moderation={moderation} />
       <RefusalModal userEmail={user.email} />