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

feat: validation des membres par un responsable #791

Open
wants to merge 64 commits into
base: main
Choose a base branch
from

Conversation

LucasCharrier
Copy link
Contributor

@LucasCharrier LucasCharrier commented Feb 3, 2025

En tant que membre je peux créer une fiche qui est validé par quelqu'un de l'équipe transverse.
En tant que membre d'un incubateur je peux créer une fiche qui est validé directement.
En tant que membre d'un incubateur je peux valider une fiche
Todo:

  • changer le text de succès après l'envoie du formulaire

@LucasCharrier LucasCharrier linked an issue Feb 3, 2025 that may be closed by this pull request
@revolunet
Copy link
Member

on a déjà l'information de l'incubateur avec les startups de la mission non ?

ca suffirait pas d'afficher la liste des incubateurs dans le form de creation ?

@LucasCharrier
Copy link
Contributor Author

LucasCharrier commented Feb 4, 2025

on a déjà l'information de l'incubateur avec les startups de la mission non ?

ca suffirait pas d'afficher la liste des incubateurs dans le form de creation ?

Alors il s'agit en effet d'un cas à la marge, le champ incubateur n'est nécessaire que s'il n'y a pas des startups. Je l'avais d'aileurs mis au début en dehors de mission. Il couvre 2 cas :

  • le cas des personnes en transverse qui n'ont pas de startup mais une mission liée à l'incubateur.
  • Et les personnes qui rejoignent beta.gouv.fr mais il n'y a pas encore de startup créée dans l'espace-membre.

Pour le premier cas j'avais en tête que c'est ce qu'on souhaitait faire depuis un moment, relier une mission transverse a un incubateur ? (mais en écrivant ces lignes je me demande si on voulait pas relier une mission a une équipe plutot ?)
Pour le deuxième cas en effet le champs pourrait être juste temporaire pour l'envoie de l'email de validation (mettre le champs incubateur en dehors de mission) mais une fois que l'email est parti on perd l'information sur le ratachement de telle personne à tel incubateur

@LucasCharrier LucasCharrier changed the title chore: add field incubator in mission chore: validation des membres par un responsable Feb 4, 2025
@LucasCharrier
Copy link
Contributor Author

on a déjà l'information de l'incubateur avec les startups de la mission non ?
ca suffirait pas d'afficher la liste des incubateurs dans le form de creation ?

Alors il s'agit en effet d'un cas à la marge, le champ incubateur n'est nécessaire que s'il n'y a pas des startups. Je l'avais d'aileurs mis au début en dehors de mission. Il couvre 2 cas :

  • le cas des personnes en transverse qui n'ont pas de startup mais une mission liée à l'incubateur.
  • Et les personnes qui rejoignent beta.gouv.fr mais il n'y a pas encore de startup créée dans l'espace-membre.

Pour le premier cas j'avais en tête que c'est ce qu'on souhaitait faire depuis un moment, relier une mission transverse a un incubateur ? (mais en écrivant ces lignes je me demande si on voulait pas relier une mission a une équipe plutot ?) Pour le deuxième cas en effet le champs pourrait être juste temporaire pour l'envoie de l'email de validation (mettre le champs incubateur en dehors de mission) mais une fois que l'email est parti on perd l'information sur le ratachement de telle personne à tel incubateur

Au moment de la validation qui est asynchrone, on a besoin de savoir si la personne qui valide à les droits. Pour avoir les droits il faut qu'elle soit membre de l'incubateur où le nouveau membre à sa mission. Il faut donc au moment de la validation être capable de récupérer l'incubateur du nouveau membre. Si ce nouveau membre n'a pas de startup, il faut donc que l'incubateur soit enregistré en bdd quelque part.

@revolunet revolunet changed the title chore: validation des membres par un responsable feat: validation des membres par un responsable Feb 7, 2025
@revolunet
Copy link
Member

Il faut donc au moment de la validation être capable de récupérer l'incubateur du nouveau membre. Si ce nouveau membre n'a pas de startup, il faut donc que l'incubateur soit enregistré en bdd quelque part.

On peut pas stocker cette info dans les paramètres de la tâche async ?

Je suis pas sûr que ce soit utile par ailleurs d'avoir cette info dans les missions ?

@LucasCharrier
Copy link
Contributor Author

LucasCharrier commented Feb 10, 2025

Il faut donc au moment de la validation être capable de récupérer l'incubateur du nouveau membre. Si ce nouveau membre n'a pas de startup, il faut donc que l'incubateur soit enregistré en bdd quelque part.

On peut pas stocker cette info dans les paramètres de la tâche async ?

Je suis pas sûr que ce soit utile par ailleurs d'avoir cette info dans les missions ?

Il faut qu'on n'en parle à l'oral. L'incubateur peut être ailleurs que dans mission, mais on a besoin de le stocker quelque part, la validation par un responsable se fait après que la tache async se soit éxécutée. C'est un endpoint d'api classique, qui s'éxécute et vérifie que la personne qui valide à les droits. Pour avoir les droits il faut regarder si elle est dans la team de l'incubateur, il faut donc avoir l'identifiant de l'incubateur, si le membre n'a pas de startup on a pas l'identifiant à moins de l'avoir enregistré.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ajouter un champs incubateur Faire une page de validation Sécurisation de l'invitation du nouveau membre
2 participants