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 : remise à plat des query params pour les vues Update #5357

Merged
merged 4 commits into from
Jan 14, 2025

Conversation

EwenKorr
Copy link
Contributor

@EwenKorr EwenKorr commented Jan 8, 2025

🤔 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.

Dans cette PR , on apporte quelques corrections, voir les commits et leur message.


Ancienne description (car cette PR était plus complexe, le reste sera fait dans une nouvelle PR)

🤔 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.

Précédemment, dans la série, nous avons introduit une vue start (#5177), qui permet d'initialiser la session du bloc et de renvoyer à la vue désirée.
Pour le moment, cette vue start n'est branchée qu'à la mise à jour de compte candidat.
Cette PR est dédiée à la généralisation de la vue start au bloc entier, c'est-à-dire que l'on branche à start les vues CheckNIRFor*, qui permettent de rechercher ou de créer un nouveau compte candidat.


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 Jan 8, 2025
@EwenKorr EwenKorr self-assigned this Jan 8, 2025
@EwenKorr EwenKorr force-pushed the ewen/job-seeker-startview branch from f56466b to 9c48f1c Compare January 9, 2025 15:52
@EwenKorr EwenKorr force-pushed the ewen/job-seeker-startview branch 4 times, most recently from 1165cf3 to 4c2ddd4 Compare January 10, 2025 12:58
@EwenKorr EwenKorr force-pushed the ewen/job-seeker-startview branch 2 times, most recently from a0f1980 to a9900a5 Compare January 10, 2025 14:33
@EwenKorr EwenKorr changed the title (wip) Création de compte candidat : généraliser la vue Start à la recherche et création de compte Création de compte candidat : généraliser la vue Start à la recherche et création de compte Jan 10, 2025
@EwenKorr EwenKorr marked this pull request as ready for review January 10, 2025 15:37
@EwenKorr
Copy link
Contributor Author

Question déploiement : j'ai supprimé la vue UpdateJobSeekerStartView et l'URL associée, pour une version générique au bloc.

Cette URL n'était utilisée que dans les deux gabarits HTML application/base.html et check_job_seeker_info_for_hire.html.

Pendant un court laps de temps, il y aura des 404 :

  • une personne affiche la page avec l'ancienne URL
  • le déploiement se fait
  • la personne clique et se prend une 404

Est-ce que c'est acceptable, ou bien je propose un bout de code temporaire pour pallier ça ?
Je pensais notamment à garder l'URL, pointer vers la nouvelle vue. Et dans la vue, ajouter 2 lignes.

@EwenKorr EwenKorr marked this pull request as draft January 10, 2025 16:28
@EwenKorr
Copy link
Contributor Author

EwenKorr commented Jan 10, 2025

CR de petite réunion :

  1. ne pas écraser la vue UpdateJobSeekerStartView, mais créer une nouvelle pour le parcours get-or-create
  2. la nouvelle vue GetOrCreateJobSeekerStartView recevrait en paramètre GET un tunnel (hire, sender, gps), le session_kind reste interne. Cette vue ne serait pas à destination des candidats. C'est le tunnel qui définira l'URL de sortie.
  3. pour le moment, garder apply.ApplyStepBaseView.init_job_seeker_session pour initialiser la session pour la vue CheckNIRForJobSeeker. À terme, cette vue (qui ne fait que vérifier qu'un candidat a un NIR) pourrait peut-être retourner dans apply.
  4. tunnel == "gps" : pour le moment, juste faire une redirection vers ?tunnel=sender&gps=true
    • à court terme, on peut imaginer ne pas passer de company dans les paramètres
    • à moyen terme, on pourrait carrément virer GPS de apply

@EwenKorr EwenKorr changed the title Création de compte candidat : généraliser la vue Start à la recherche et création de compte Création de compte candidat : une vue Start pour la recherche et création de compte Jan 13, 2025
@EwenKorr EwenKorr force-pushed the ewen/job-seeker-startview branch from a9900a5 to 198b305 Compare January 13, 2025 15:53
@EwenKorr EwenKorr marked this pull request as ready for review January 13, 2025 15:53
@EwenKorr EwenKorr force-pushed the ewen/job-seeker-startview branch 4 times, most recently from d311739 to b4c14e8 Compare January 13, 2025 16:39
itou/www/job_seekers_views/views.py Outdated Show resolved Hide resolved
itou/www/apply/views/submit_views.py Outdated Show resolved Hide resolved
itou/www/job_seekers_views/views.py Outdated Show resolved Hide resolved
itou/www/job_seekers_views/views.py Outdated Show resolved Hide resolved
@EwenKorr EwenKorr force-pushed the ewen/job-seeker-startview branch from b4c14e8 to c771553 Compare January 14, 2025 07:48
@EwenKorr EwenKorr force-pushed the ewen/job-seeker-startview branch from c771553 to b933ad4 Compare January 14, 2025 09:24
@EwenKorr EwenKorr changed the title Création de compte candidat : une vue Start pour la recherche et création de compte Création de compte candidat : remise à plat des query params pour les vues Update Jan 14, 2025
In a Hire process, the start URL is `apply:start_hire`
In Update views, we don't need the company, but for the sake of clarity
we now need a `from_url` so that this value can be set in one less place.
@EwenKorr EwenKorr force-pushed the ewen/job-seeker-startview branch from b933ad4 to b975a76 Compare January 14, 2025 09:29
@EwenKorr EwenKorr requested a review from xavfernandez January 14, 2025 09:32
@@ -145,7 +145,8 @@ def init_job_seeker_session(self, request):
request.session,
data={
"config": {
"reset_url": self.get_reset_url(),
"from_url": self.get_reset_url(),
"session_kind": "job-seeker-get-or-create-job-seeker",
Copy link
Contributor

Choose a reason for hiding this comment

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

Ce n'est pas vraiment un get-or-create mais plutôt un check-nir-job-seeker :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

La version longue pour être cohérent job-seeker-check-nir-job-seeker ou bien juste job-seeker-check-nir ?
J'ai une préférence pour la cohérence, mais il n'y en a techniquement pas besoin.

Copy link
Contributor

@xavfernandez xavfernandez Jan 14, 2025

Choose a reason for hiding this comment

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

Comme tu préfères, tu sais ce que je pense de cette session 😛 🪓

itou/www/job_seekers_views/views.py Outdated Show resolved Hide resolved
migrations.md Outdated Show resolved Hide resolved
@EwenKorr EwenKorr force-pushed the ewen/job-seeker-startview branch from 2a7633b to 5266f66 Compare January 14, 2025 09:41
@EwenKorr EwenKorr added this pull request to the merge queue Jan 14, 2025
Merged via the queue into master with commit 27abaa9 Jan 14, 2025
9 checks passed
@EwenKorr EwenKorr deleted the ewen/job-seeker-startview branch January 14, 2025 12:16
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.

2 participants