added 1 commit

• aca3a2c6 - re-allow superadmin impersonating

Compare with previous version

changed title from allow superadmin to delete users in tav env + fix impersonating + return to… to allow superadmin to delete users in tav env + fix impersonating

Pour moi c'est OK, même si philosophiquement ça me paraît bizarre que le rôle adhérent ait un statut spécial par rapport aux autres rôles. Par exemple, le fait que la suppression d'un utilisateur supprime l'adhérent et pas le prestataire.

D'autre part, comme le montrent les deux screenshots dans le ticket, la suppression échoue dès lors qu'il y a au moins un flux qui concerne le user (ou l'adhérent ?). Du coup cette fonctionnalité va être assez limitée.

Si c'est OK pour toi tu peux merger.

Pour moi c'est OK, même si philosophiquement ça me paraît bizarre que le rôle adhérent ait un statut spécial par rapport aux autres rôles. Par exemple, le fait que la suppression d'un utilisateur supprime l'adhérent et pas le prestataire.

Pour moi c'est normal, car un compte adhérent est en OneToOne avec User (un compte Adhérent n'existe pas sans un compte User), alors qu'un compte Prestataire par exemple peut être géré par plusieurs utilisateurs (OneToMany).

D'autre part, comme le montrent les deux screenshots dans le ticket, la suppression échoue dès lors qu'il y a au moins un flux qui concerne le user (ou l'adhérent ?). Du coup cette fonctionnalité va être assez limitée.

Après discution avec Charles, ce n'est pas gênant pour l'utilisation prévue. Je vais ajouter un message avertissant qu'un utilisateur dont l'adhérent a des flux associés ne pourra pas être supprimé.

En relisant les attentes de Charles, je vois aussi qu'il souhaite que :

"Que l'utilisateur soit du Groupe Adhérent, Prestataire, Comptoir... l'utilisateur ne devra plus se retrouver dans la liste relative à son groupe."

Du coup, peut-être faut-il d'abord vérifier avec lui que c'est OK de ne pas aller supprimer les autres rôles ?

C'est ok pour ça, c'est bien les comptes utilisateurs qui doivent être supprimés.

added 1 commit

• 20f4c1e3 - user admin: prevent deleting user if its adherent account has flux

Compare with previous version

Même si aucun flux n'est lié à l'adhérent, il peut y avoir un flux lié au user, par exemple dans le cas d'une cotisation, ou bien, plus réaliste pour nous, dans le cas d'un achat de monnaie adhérent (vérifié sur prod tav : c'est le format classique d'un flux de type achat de monnaie).

ok, il faut donc bloquer si un quelconque flux est lié à l'adhérent ou à l'utilisateur

changed this line in version 5 of the diff

corrigé

added 216 commits

• 20f4c1e3...b3184aff - 215 commits from branch develop
• 70f6b7d9 - merge conflicts

Compare with previous version

added 1 commit

• 89114128 - prevent deleting user if has flux to its account OR its adherent account

Compare with previous version

merged

mentioned in commit e6e07f16