Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Faciliter l'ajout d'utilisateur #457

Closed
2 tasks done
IGNF-Xavier opened this issue Jun 28, 2024 · 8 comments
Closed
2 tasks done

Faciliter l'ajout d'utilisateur #457

IGNF-Xavier opened this issue Jun 28, 2024 · 8 comments
Assignees
Labels
enhancement Nouvelle fonctionnalité ou amélioration

Comments

@IGNF-Xavier
Copy link

IGNF-Xavier commented Jun 28, 2024

LE mail de demande d'ajout est reçu

Image

  • Solution de contournement : renvoyer vers le tutoriel pour l'ajout d'utilisateur depuis ce mail
  • Solution cible : lien qui permet d'aller sur la modale d'ajout préremplie avec l'uuid utilisateur (en passant par étape de connexion si session inactive)
@IGNF-Xavier IGNF-Xavier added this to the MVP-0.2_September2024 milestone Jun 28, 2024
@IGNF-Xavier IGNF-Xavier added the enhancement Nouvelle fonctionnalité ou amélioration label Jun 28, 2024
@slafayIGN
Copy link
Contributor

Le lien d'ajout direct a pourtant déjà été prévu dans #222 (et #133) :
image
Un lien de cette forme mène à la page de gestion des membres avec la modale d'ajout ouverte et le champ Identifiant de l'utilisateur rempli : /entrepot/{uuid-entrepot}/membres?userId={uuid-utilisateur}

Il reste à l'ajouter dans le template de mail envoyé à l'adresse de contact de la communauté : https://github.com/IGNF/cartes.gouv.fr/blob/45f92a1ed979186bb0f66a3eb43e9a96758b4a2e/templates/Mailer/join_community.html.twig

@slafayIGN slafayIGN assigned pprev94 and unassigned slafayIGN Jul 1, 2024
@pprev94
Copy link
Contributor

pprev94 commented Jul 4, 2024

@IGNF-Xavier Peux-tu préciser le contenu du mail stp ? Merci

@IGNF-Xavier
Copy link
Author

Bonjour,

Un utilisateur demande à rejoindre "DEMO_IGN_SPP" (id=33c2c13d-353d-4d88-a557-19e90e741841)

Informations sur le demandeur :
Adresse mail : azerty@ign.fr
Nom : ERTY
Prénom : Az
Identifiant : XXX-613234-yyyy-fdgndfg-shnsf542
Date de la demande : 01-07-2024 13:39
Message :
Message | Pas de message

En tant que superviseur de "DEMO_IGN_SPP", vous pouvez accepter ou rejeter sa demande d'adhésion en allant sur la page de suivante.

Cordialement, l’équipe de cartes.gouv.fr

@pprev94
Copy link
Contributor

pprev94 commented Jul 11, 2024

Salut @IGNF-Xavier

Il n'est pas possible d'ajouter ce lien (/entrepot/{uuid-entrepot}/membres?userId={uuid-utilisateur}) dans l'email pour la raison suivante :
Les données d'une communauté dans la liste des communautés publiques retournées par la route de l'API GET /catalogs/communities contient uniquement :

{ "name": "Communauté Simalis NL 2024", "technical_name": "simalis_nl_2024", "contact": "deleted-96645ae5-d375-4f12-8c3a-1a22d25c6023@no_reply.ign.fr", "_id": "7a35fc6c-4978-4da0-abc9-196640dcf42f", "public": true }

Il manque le datastoreId

@IGNF-Xavier
Copy link
Author

@pprev94 Est-ce que la résolution de IGNGPF-2799 permettrait d'étendre les champs recherchés au datastoredId et résoudre ce problème?

@pprev94
Copy link
Contributor

pprev94 commented Jul 11, 2024

Sauf si on ne veut pas exposer le datastore pour des raisons de sécurité ou autres

@slafayIGN slafayIGN added the blocked Nécessite une évolution d'API, de config d'environnement, ou la fourniture de contenu label Aug 14, 2024
@slafayIGN
Copy link
Contributor

slafayIGN commented Aug 14, 2024

Techniquement, une communauté peut ne pas être liée à un datastore. Et pourtant, on pourrait toujours y ajouter des membres. Il y a donc un problème avec notre conception de départ qui ne visait que les workflows d'alimentation-diffusion et basait toutes les urls sur l'id du datastore (dont https://cartes.gouv.fr/entrepot/{datastoreId}/membres). Baser les urls de gestion des membres sur la communauté est plus logique mais impose un refactoring de cette partie.

@ocruze ocruze assigned ocruze and unassigned pprev94 Aug 20, 2024
@ocruze ocruze removed their assignment Sep 11, 2024
@slafayIGN slafayIGN removed the blocked Nécessite une évolution d'API, de config d'environnement, ou la fourniture de contenu label Sep 11, 2024
@AlexandreJaqua
Copy link

J'ai reçu une demande pour rejoindre une commu donc j'ai pu valider tout ça, c'est impec. Infos sur le demandeur + date de la demande + message + lien direct vers la modale d'ajout (et en bonus, si on clique en étant déconnecté, après la page de connexion on tombe bien dessus). Je ferme !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Nouvelle fonctionnalité ou amélioration
Projects
None yet
Development

No branches or pull requests

5 participants