Nouvelle page des gestion des membres d'une collectivité #3227
Conversation
marc-rutkowski
commented
Jun 4, 2024
•
marc-rutkowski
commented
- Envoi automatisé des invitations à rejoindre la collectivité.
- Pagination de la liste des membres.
- Nouvel habillage graphique.
- Corrige un problème qui empêchait la suppression d'un membre que l'on venait d'ajouter (compte pré-existant)
| const membre_id = membre.user_id; | ||
| if (!membre_id) { | ||
| return; | ||
| } |
There was a problem hiding this comment.
Je vois que ça vient de la définition de l'interface Membre :
export interface Membre {
email: string;
user_id: string | null;
…
}
mais est ce que tu sais pourquoi la propriété user_id n'est toujours définie (i.e. non null) ?
There was a problem hiding this comment.
C'est pour le cas d'un membre invité : il est dans la table des invitations mais pas encore dans la table des utilisateurs.
En l'occurence le sélecteur pour changer le niveau d'accès d'un membre invité n'est même pas affiché donc effectivement le typage pourrait être plus strict ici mais est-ce vraiement nécessaire ?
Je te propose de laisser ça comme ça.
| @@ -65,53 +112,59 @@ export const Membres = ({ | |||
| isLoading={isLoading} | |||
| updateMembre={updateMembre} | |||
| removeFromCollectivite={removeFromCollectivite} | |||
| sendInvitation={sendInvitation} | |||
There was a problem hiding this comment.
Pour comprendre, dans quel cas on veut pouvoir envoyer l'invitation seule, hors du processus d'ajout d'un membre à la collectivité ?
There was a problem hiding this comment.
C'est pour le bouton "renvoyer l'invitation".
