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

Création de compte candidat : nettoyage des URLs et vues dépréciées #5180

Merged
merged 2 commits into from
Dec 3, 2024

Conversation

EwenKorr
Copy link
Contributor

@EwenKorr EwenKorr commented Dec 2, 2024

🤔 Pourquoi ?

Dans l'optique de Créer un compte candidat depuis l'espace Mes candidats, on Extrait la création de compte candidat du parcours de candidature.

Nous sommes lundi. Dans la nuit, un cron a lancé une commande pour nettoyer les sessions, on peut donc être sûr que les nouvelles sessions ne contiennent plus les URLs dépréciées. Dans cette PR, on supprime le code de :

  • CheckNir
  • SearchByEmail
  • CreateJobSeeker

Les autres étapes : https://www.notion.so/plateforme-inclusion/Extraire-le-parcours-de-cr-ation-de-compte-candidat-130e8fa5c35b80b9947cea2573cf90e7?pvs=4#130e8fa5c35b800b966fdd4722014657

@EwenKorr EwenKorr added the no-changelog Ne doit pas figurer dans le journal des changements. label Dec 2, 2024
@EwenKorr EwenKorr self-assigned this Dec 2, 2024
On Sunday the sessions were cleaned, so we are assured that there
is no reference to deprecated URLs. We're deleting deprecated URLs and
views related to:
- CheckNir
- SearchByEmail
@EwenKorr EwenKorr force-pushed the ewen/compte-candidat-clean branch from 2be344d to 35914c8 Compare December 2, 2024 14:27
@EwenKorr EwenKorr marked this pull request as ready for review December 2, 2024 14:28
Copy link
Contributor

@francoisfreitag francoisfreitag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹

J’ai fait une petite passe sur les vues restantes, au cas où... 👌

Copy link
Contributor

@xavfernandez xavfernandez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙏

Normalement tu devrais aussi pouvoir te débarrasser de

# TODO(ewen): temporary condition to fill self.company when not using the new session system
if company_pk := kwargs.get("company_pk"):
self.job_seeker_session.set("apply", {"company_pk": company_pk} | self.job_seeker_session.get("apply", {}))
?

@EwenKorr
Copy link
Contributor Author

EwenKorr commented Dec 3, 2024

Oui, en effet. J'hésitais, j'avais peur que ça casse quelque chose quand la PR #5177 sera fusionnée.
Mais on aura deux jeux de vues distincts (les nouvelles avec la bonne session, et les dépréciées avec un company_pk dans l'URL). Pas de conflit donc.

Même chose pour les dernières vues qui resteront, CheckJobSeekerInformation : a priori le fonctionnement va changer, on va injecter une session, donc je repasserai par des vues dépréciées et une PR de ménage 😁

On Sunday the sessions were cleaned, so we are assured that there
is no reference to deprecated URLs. We're deleting deprecated URLs and
cleaning CreateJobSeeker* views.
@EwenKorr EwenKorr force-pushed the ewen/compte-candidat-clean branch from 35914c8 to 256b2b1 Compare December 3, 2024 07:32
@EwenKorr EwenKorr added this pull request to the merge queue Dec 3, 2024
Merged via the queue into master with commit 46c6f95 Dec 3, 2024
11 checks passed
@EwenKorr EwenKorr deleted the ewen/compte-candidat-clean branch December 3, 2024 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog Ne doit pas figurer dans le journal des changements.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants