-
Notifications
You must be signed in to change notification settings - Fork 24
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
Conversation
f56466b
to
9c48f1c
Compare
1165cf3
to
4c2ddd4
Compare
a0f1980
to
a9900a5
Compare
Start
à la recherche et création de compteStart
à la recherche et création de compte
Question déploiement : j'ai supprimé la vue Cette URL n'était utilisée que dans les deux gabarits HTML Pendant un court laps de temps, il y aura des 404 :
Est-ce que c'est acceptable, ou bien je propose un bout de code temporaire pour pallier ça ? |
CR de petite réunion :
|
Start
à la recherche et création de compteStart
pour la recherche et création de compte
a9900a5
to
198b305
Compare
d311739
to
b4c14e8
Compare
b4c14e8
to
c771553
Compare
c771553
to
b933ad4
Compare
Start
pour la recherche et création de compteIn 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.
b933ad4
to
b975a76
Compare
itou/www/apply/views/submit_views.py
Outdated
@@ -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", |
There was a problem hiding this comment.
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
:)
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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 😛 🪓
2a7633b
to
5266f66
Compare
🤔 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 vuesCheckNIRFor*
, 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