Vie privée : intégrer le formulaire de demande de régularisation NIR dans les Emplois à la place de Tally [GEN-2516]

[EwenKorr]

Premiers commits dans les PR #6428 et #6429 (qui seront closes une fois que les commits seront validés là-bas)

🤔 Pourquoi ?

Il existe depuis longtemps un formulaire Tally pour faire les demandes de régularisation du NIR qui a été mis en place pour des raisons de simplicité. Le NIR étant une donnée sensible, nous devons éviter au max d’en laisser la trace dans d’autres outils et nous devons respecter les délais légaux de conservation.

Le supprimer de Tally n’est pas suffisant car il reste dans l’API qui le transmet ensuite à Zendesk. Nous n’avons pas la main pour le retirer de l’API.

Besoin : ne plus faire transiter le NIR par Tally (ni par Zendesk si on veut être vraiment carré) et intégrer le formulaire de demande sur les Emplois directement.

🍰 Comment ?

Après #6428 (création du modèle et de la vue admin), #6429 (formulaire, url, vue, et notification email), cette présente PR modifie le lien sous le champ NIR, pour pointer vers le formulaire local et non plus vers Tally.

[notion-workspace]

🌿 Intégrer le formulaire de demande de régularisation du NIR dans les Emplois à la place de Tally

[github-actions]

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

[vincentporte]

🌟 Belle série de PR Erwan, Congrats !
Il y a moyen de faire pas mal de simplification en supprimant la reference à la candidature, puisque le NIR est attaché au candidat

[tonial]

Quand tu as des PRs qui se suivent, tu peux changer dans github la base des PRs pour qu'on n'y trouve que les commits qui la concernent
Cliques sur EDIT à droite du titre de la PR, puis sur le dropdown base: master

[EwenKorr]

Quand tu as des PRs qui se suivent, tu peux changer dans github la base des PRs pour qu'on n'y trouve que les commits qui la concernent Cliques sur EDIT à droite du titre de la PR, puis sur le dropdown base: master

Yep, merci. En l'occurrence, ce n'est pas trop tard maintenant ?

Et ce serait de quelle PR à quelle PR ? Des premières à la dernière qui serait fusionnée ?

[leo-naeka]

Non ce n'est pas trop tard, sachant que tu rebase chacune de tes PRs entre elles, ça fait sens même ;)

• La base de ta PR 1 est master
• Base de PR 2 est PR 1
• Base de PR 3 est PR 2

Une fois les 3 PR validées, tu pourras :

• Close PR 1 et 2
• Squash les 3+ commits sur PR 3
• Changer la base de PR 3 en master (uniquement si le fait d'avoir close PR 2 ne l'a pas automatiquement changé, je pense que cette étape va sauter, tu nous diras)
• Ajouter à la merge queue

---

Par rapport à ton screenshot c'est logique que les commits des autres branches soient retirés. Et normalement aucun de nous n'a commenté de changements de PR 1 ou PR 2 sur cette PR. Mais si tel était le cas, ils seraient effectivement résolus car outdated.

[github-actions]

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

[EwenKorr]

Changer la base de PR 3 en master (uniquement si le fait d'avoir close PR 2 ne l'a pas automatiquement changé, je pense que cette étape va sauter, tu nous diras)

Pour info, clore une PR ne suffit visiblement pas à changer la cible de la fusion.

[tonial]

Tu ne voulais pas merger les PR les unes après les autres ?

[EwenKorr]

Tu ne voulais pas merger les PR les unes après les autres ?

Au début, oui, mais on m'a fait remarquer que ça ne fait pas sens d'avoir plusieurs commits dans master (si on git bisect par exemple, il vaut mieux avoir tous les ajouts sous un seul commit). Le temps de la relecture par contre, ça aide bien.

[tonial]

Tu comptes squasher les commits ?

[EwenKorr]

Oui :D

et https://gip-inclusion.slack.com/archives/C0412CTV63D/p1751370772253999?thread_ts=1751360388.417249&cid=C0412CTV63D

[EwenKorr]

Je regarde demain deux derniers trucs (en voyant passer d'autres PR) :

• m'assurer que le support a accès aux demandes de régul dans l'admin
• tester des fusions de compte et voir ce qui casse (et si ça casse, ce sera plutôt pour une autre PR)

[EwenKorr]

Je regarde demain deux derniers trucs (en voyant passer d'autres PR) :

* m'assurer que le support a accès aux demandes de régul dans l'admin

C'est modifié.

* tester des fusions de compte et voir ce qui casse (et si ça casse, ce sera plutôt pour une autre PR)

Ça fonctionne :

• fusionner un compte pro modifie bien la FK requested_by
• transférer un compte candidat ne modifie rien (et je pense que ce n'est pas gênant pour les demandes de modification de NIR)