Produits Entr'ouvert » Authentic 2

C'est pas mal lié à #50054, où on a décidé d'ajouter un champ de type liste simple avec des choix qui englobent plusieurs types d'évènement, pour s'éviter une liste à choix multiple.
Donc sans revenir sur ce choix ici tout ce qu'on peut faire c'est comme tu l'écris ajouter la liste des évènements. Comme elle est très longue on est obligés de l'afficher dans une popup, ça donnerait un truc comme ça.

En pratique il n'y a aucun choix dans la liste déroulante qui me semblait évident pour chercher les suppressions :utilisateurs / modification de profil peut-être, puis filtrer sur backoffice / gestion des utilisateurs, et dans le tas de toute façon en venir au ctrl-f pour tenter de trouver une suppression dans le tableau, et passer à côté de la suppression réalisée via l'API.

Aidé par la boite de dialogue de la capture, j'essaie désormais event:deletion et ça me capte bien tout y compris les suppressions des rôles, j'essaie event:user.deletion en me disant que je ne veux pas uniquement les suppressions réalisées par le backoffice mais ça ne ne matche alors pas celle du backoffice, pour avoir celles-ci, il faut event:manager.user.deletion (et ça m'attrape les suppressions via l'API).

Je trouve la popup utile mais elle ne va pas répondre totalement à ma demande de pouvoir chercher tous les événements de suppression d'utilisateurs.

Valentin Deniaud a écrit :

• en plus de la popup, faire en sorte que « event:xxx » soit moins strict et puisse matcher xxx_yyy

À la réflexion je trouve cette idée pas ouf, on risque de passer de « xxx ne matche pas xxx_yyy » à « xxx matche xxx_yyy alors que je voulais juste xxx ».

• indépendamment de la popup, ajouter un groupe « Général » dans la liste déroulante et y mettre « Suppression d'un utilisateur »

J'ajoute un patch pour faire ça.

Benjamin Dauvergne a écrit :

Pour l'instant le seul retour qu'on a c'est exactement l'inverse, peut-être as-tu un contre exemple plus précis ?

Non pas vraiment, aller encore un patch qui ne mange pas de pain.

Valentin Deniaud a écrit :

Non pas vraiment, aller encore un patch qui ne mange pas de pain.

Ça mangeait un peu de pain, il fallait des adaptations aux tests.

Quand c'est vert.

commit 4b485158e1708798fa756e3899094d47b3daee7a
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Tue Aug 24 10:25:08 2021 +0200

    journal: make event type search less strict (#55969)

commit 469e46412e6c0f2ccbfd5e18be0aea13645d1cb9
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Mon Aug 23 16:20:27 2021 +0200

    manager: add user deletions choice in journal search form (#55969)

commit 167d5ebba694c20a6abcd31331b1f15fa64d136d
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Wed Aug 4 16:10:14 2021 +0200

    manager: add access to journal event types list (#55969)

commit d23b49306e945fce9513a2d0412a8b564d22cc42
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Wed Aug 4 16:09:08 2021 +0200

    journal: refactor journal search documentation (#55969)