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 : une vue Start pour la recherche et la création de compte #5384

Merged
merged 2 commits into from
Jan 16, 2025

Conversation

EwenKorr
Copy link
Contributor

🤔 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 pour les vues Update* (#5177), qui permet d'initialiser la session du bloc et de renvoyer à la vue désirée.
Cette PR est dédiée à la réalisation du même travail poru les vues relatives à la recherche et la création de compte candidat. On utilise une nouvelle vue Start pour initialiser la session avant d'entrer dans le bloc par la vue CheckNIRForSenderView, qui est la première étape du processus.

La vue CheckNIRForJobSeekerView, qui n'est utilisée par le candidat que pour insérer son NIR, est laissée en l'état.

Concernant GPS, les premières briques sont posées pour déconnecter la création de bénéficiaire de l'application apply.


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 14, 2025
@EwenKorr EwenKorr self-assigned this Jan 14, 2025
@EwenKorr EwenKorr marked this pull request as draft January 14, 2025 10:21
@EwenKorr EwenKorr force-pushed the ewen/job-seeker-get-or-create-start branch 3 times, most recently from 31aae0e to c836ab5 Compare January 14, 2025 12:56
@EwenKorr EwenKorr marked this pull request as ready for review January 14, 2025 12:58
@EwenKorr EwenKorr requested a review from xavfernandez January 14, 2025 12:58
itou/www/job_seekers_views/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 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
itou/www/job_seekers_views/views.py Show resolved Hide resolved
data = {
"config": {
"from_url": self.from_url,
"session_kind": self.session_kind,
Copy link
Contributor

Choose a reason for hiding this comment

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

Je verrais bien l'info du tunnel dans la session, notamment pour gps.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

D'accord, donc la logique derrière c'est :

  • la session_kind permet de vérifier qu'on fait bien l'action (Update, GetOrCreate, CheckNirForJobSeeker) pour laquelle la session a été créée, quel que soit le tunnel
  • le tunnel donne une information supplémentaire dans le cas de GetOrCreate (et peut-être bientôt de Update) : est-ce qu'on est dans le contexte d'une candidature, embauche ou création de bénéficiaire GPS. Cette info change la cosmétique ("bénéficiaire" pour GPS, "candidat" pour une candidature…) et permettra de savoir où on sort une fois le candidat créé (je m'y attèlerai dans une autre PR)

Copy link
Contributor

Choose a reason for hiding this comment

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

Ça serait l'idée oui. Mais effectivement, je me demande si ça ne serait finalement pas plus simple de n'avoir qu'une seule clef session_kind pour faire les deux. Peut-être que @francoisfreitag a un avis ?

Copy link
Contributor

Choose a reason for hiding this comment

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

Botte en touche. Tout dépend de ce qu’on souhaite modifier. J’ai l’impression que pour le moment on pourrait avoir les deux séparés, pour ne pas avoir à gérer le produit cartésien session_kind et tunnel. Jusqu’au moment où on voudra un cas particulier différemment un tunnel. 🙈

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Merci pour vos retours.
Gardons les deux séparés alors !

itou/www/job_seekers_views/enums.py Show resolved Hide resolved
itou/www/job_seekers_views/views.py Outdated Show resolved Hide resolved
itou/www/apply/views/submit_views.py Outdated Show resolved Hide resolved
@EwenKorr EwenKorr force-pushed the ewen/job-seeker-get-or-create-start branch from ba3c002 to c1d1675 Compare January 16, 2025 10:40
@EwenKorr
Copy link
Contributor Author

Il y a les dispatch de JobSeekerForSenderBaseView et de CheckNIRForJobSeekerView (https://github.com/gip-inclusion/les-emplois/blob/master/itou/www/job_seekers_views/views.py#L270) qu'on pourrait virer aussi, vu que c'est le session_kind qui s'en occupe (et qu'on voit dans les logs que ces dispatchs ne sont jamais lus).

Mais il y a un peu de travail à faire sur les tests, donc je me le note pour une autre PR.

itou/www/apply/views/submit_views.py Outdated Show resolved Hide resolved
@EwenKorr EwenKorr force-pushed the ewen/job-seeker-get-or-create-start branch from 6692660 to a241f7a Compare January 16, 2025 12:46
Set 3 possible session_kinds:
- UPDATE (for sender/hire only, for now)
- GET_OR_CREATE (for sender only)
- CHECK_NIR_JOB_SEEKER (for job_seekers only)
The `GetOrCreateJobSeekerStartView`'s role is to initialize a session
before redirecting to `CheckNIRForSenderView`, which is the first
step in the get-or-create job seeker process.

The process for job seekers themselves (CheckNIRForJobSeeker) is left untouched.

We define 3 tunnels:
- sender (a regular apply process)
- hire
- gps
@EwenKorr EwenKorr force-pushed the ewen/job-seeker-get-or-create-start branch from a241f7a to a801c59 Compare January 16, 2025 12:46
@EwenKorr EwenKorr added this pull request to the merge queue Jan 16, 2025
Merged via the queue into master with commit 582766b Jan 16, 2025
9 checks passed
@EwenKorr EwenKorr deleted the ewen/job-seeker-get-or-create-start branch January 16, 2025 13:12
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