Candidature : postuler pour un candidat depuis l'espace "Mes candidats" #4860

EwenKorr · 2024-10-01T15:11:46Z

🤔 Pourquoi ?

ETQU, lorsque je clique sur le bouton postuler depuis une ligne de la liste candidat ou depuis une page de détail d’un candidat,

je peux faire une recherche d’emploi et postuler sans avoir à renseigner l’adresse e-mail, le NIR du candidat ou les informations personnelles du candidat.

🍰 Comment ?

On passe l'identifiant public du candidat en paramètre GET (job_seeker=UUID).
Une mixin permet d'ajouter aux vues utilisées la variable job_seeker dans le contexte, si l'utilisateur connecté a la permission (user.can_view_personal_info). Quand cette variable existe, les URLs utilisées contiennent paramètre GET, et un bandeau d'information rappelle que l'on postule pour un candidat.

🚨 À vérifier

Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ Comment tester

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

notion-workspace · 2024-10-01T15:12:13Z

🌿 [3/5] En tant qu’utilisateur je peux postuler pour l’un de mes candidat depuis l’espace “Mes candidats”

github-actions · 2024-10-04T10:30:28Z

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

EwenKorr · 2024-10-04T12:07:35Z

itou/templates/companies/card.html

+            <p class="mb-0 text-start w-100">
+                <i class="ri-arrow-right-line pe-2"></i><strong>Vous postulez actuellement pour {{ job_seeker.get_full_name }}</strong>
+            </p>
+            <a href="{% if request.user.is_prescriber %}{% url "job_seekers_views:list" %}{% else %}{% url "apply:list_prescriptions" %}{% endif %}"


Quitter :
Si prescripteur : retour à la liste des candidats
Si employeur : retour à la liste des orientations

EwenKorr · 2024-10-04T12:24:27Z

itou/fixtures/django/05_test_users.json

+  {
+    "fields": {
+      "address_filled_at": null,


Une utilisatrice en plus, orientée par une entreprise vers une autre entreprise
(ça permet de tester des choses là : http://127.0.0.1:8000/apply/prescriptions/list)

EwenKorr · 2024-10-04T12:28:16Z

itou/templates/companies/card.html

@@ -32,7 +49,7 @@ <h2 class="visually-hidden">Actions rapides</h2>
                    </div>
                {% else %}
                    <div class="form-group col-12 col-lg-auto">
-                        <a href="{% url 'apply:start' company_pk=siae.pk %}"
+                        <a href="{% url 'apply:start' company_pk=siae.pk %}{% if job_seeker %}?job_seeker={{ job_seeker.public_id }}{% endif %}"


Pas mal d'ajouts de ?job_seeker= dans les liens jusqu'à apply:start.

EwenKorr · 2024-10-04T12:32:35Z

itou/www/apply/views/submit_views.py

@@ -245,6 +245,37 @@ def get(self, request, *args, **kwargs):
            else:
                self.apply_session.init({"selected_jobs": [job_description.pk]})

+        # Go directly to step ApplicationJobsView if we're carrying the job seeker public id with us.
+        if job_seeker_public_id := request.GET.get("job_seeker"):


On vérifie ici si on a job_seeker= passé en paramètre.
On vérifie s'il est valide, s'il correspond à un candidat et si l'utilisateur actuel peut voir ses informations personnelles.

A priori, si on peut "Postuler pour ce candidat", on voit ses informations personnelles ?
Je ne suis pas sûr, cela dit.

je crois bien qu'on peut postuler pour un candidat sans voir ses infos (son nom sera masqué notamment)

EDIT: oui.
Tu peux aller sur la demo, te connecter avec le compte d'orienteur et postuler pour le nir 201105918336235

Ah mais oui ! Et puis je vois que l'espace "Mes candidats" comporte des candidats sans infos personnelles.

- added a link in job seekers list ("Mes candidats") to apply in the name of a specific job seeker - all the links up to `apply:start` have been updated with a GET param (`job_seeker={public_id}`), this allows to keep track of the job seeker we are applying for - when a `job_seeker` GET param is present, and when the current `user` is allowed to, an alert is displayed at the top of the page to remind that we're applying for this job seeker

EwenKorr · 2024-10-04T12:36:45Z

itou/www/apply/views/submit_views.py

+        # Create an empty variable to avoid unknown variable template errors in tests
+        context["job_seeker"] = None


Bonne pratique ou pas ?

Je trouve ça mieux que de devoir mettre des default partout dans les templates en tout cas :)

EwenKorr · 2024-10-04T12:38:22Z

tests/www/job_seekers_views/test_list.py

+                    '<a class="btn btn-ico btn-link" data-bs-toggle="tooltip" data-bs-placement="top" '
+                    'title="Postuler pour ce candidat" '
+                    f'href="{reverse("search:employers_results")}?job_seeker={public_id}">'


Peut-être un peu trop de markup ?

EwenKorr · 2024-10-04T12:39:50Z

tests/www/search/tests.py

+            f'<a href="{reverse("search:job_descriptions_results")}?city={guerande.slug}'
+            f'&job_seeker={job_seeker_public_id}" '
+            'class="btn btn-ico btn-dropdown-filter" aria-label="Réinitialiser les filtres actifs">'
+            '<i class="ri-eraser-line fw-bold" aria-hidden="true"></i>'
+            '<span>Effacer tout</span></a>'
+        )


Peut-être beaucoup trop de markup ?
Le moindre changement de classe fait capoter le test…
Mais en même temps je voudrais être sûr que le bouton "Effacet tout" contient bien l'attribut href avec job_seeker.

EwenKorr · 2024-10-04T12:43:59Z

tests/www/apply/test_submit_from_job_seekers_list.py

Un nouveau fichier de test, séparé du reste (vu que tout va être séparé par la suite).

tonial

Ça m'a l'air très efficace comme solution :)

Le mixin mérite d'être retravaillé je pense pour gérer la gestion de l'argument au début et non à la fin du traitement de la requête

tonial · 2024-10-04T13:10:26Z

itou/templates/companies/job_description_card.html

+    {% if job_seeker %}
+        <div class="alert alert-primary fade show " role="status">
+            <p class="mb-0 text-start w-100">
+                <i class="ri-arrow-right-line pe-2"></i><strong>Vous postulez actuellement pour {{ job_seeker.get_full_name }}</strong>
+            </p>
+            <a href="{% if request.user.is_prescriber %}{% url "job_seekers_views:list" %}{% else %}{% url "apply:list_prescriptions" %}{% endif %}"
+               class="btn btn-link-white btn-ico position-absolute top-0 end-0"
+               aria-label="Quitter">
+                <i class="ri-close-line ri-lg pe-2" aria-hidden="true"></i>
+                Quitter
+            </a>
+        </div>
+    {% endif %}


c'est présent plusieurs fois : tu devrais le mettre dans un template séparé et l'importer à chaque fois avec un include

tonial · 2024-10-04T13:12:04Z

itou/templates/search/includes/siaes_search_content.html

@@ -7,7 +7,13 @@
                        <div class="col-12">
                            {% include "search/includes/siaes_search_subtitle.html" %}
                            <div class="d-block w-100">
-                                <form hx-get="{{ request.path }}" hx-trigger="change delay:.5s" hx-include="#id_city" hx-indicator="#job-search-results" hx-target="#job-search-results" hx-swap="outerHTML" hx-push-url="true">
+                                <form hx-get="{{ request.path }}{% if job_seeker %}?job_seeker={{ job_seeker.public_id }}{% endif %}"


tu as besoin de transmettre le job_seeker dans hx-get ?

tonial · 2024-10-04T13:13:16Z

itou/www/apply/views/submit_views.py

+        # Create an empty variable to avoid unknown variable template errors in tests
+        context["job_seeker"] = None


Je trouve ça mieux que de devoir mettre des default partout dans les templates en tout cas :)

tonial · 2024-10-04T13:24:45Z

itou/www/apply/views/submit_views.py

+                    .select_related("jobseeker_profile")
+                    .first()
+                )
+            except ValidationError:


Je pense que si le job_seeker n'existe pas (le public_id n'est pas valide) il faudrait lever une 404, ou ou rediriger avec une erreur.

Du coup j'aurais tendance à traiter le query params job_seeker dans le setup() du mixin (ou dans le dispatch() peut-être, j'ai un doute) et à l'enregistrer dans self.job_seeker
Dans get_context_data() tu n'auras qu'à ajouter job_seeker au context s'il a été trouvé dans le setup()

tonial · 2024-10-04T13:28:42Z

tests/www/companies_views/test_card_views.py

+        # If anonymous user, do not show the alert (meaning the context does not contain the job_seeker)
+        response = self.client.get(url)
+        self.assertNotContains(
+            response, f"Vous postulez actuellement pour {job_application.job_seeker.get_full_name()}"


Tu devrais définir cette variable une fois au début du test, et la réutiliser par la suite :
BANNER_TXT = f"Vous postulez actuellement pour {job_application.job_seeker.get_full_name()}"

Comme ça, tu auras des assertContains(response, BANNER_TXT) et assertNotContains(response, BANNER_TXT) ce qui t'assures que le jour où ça change, tu modifies bien tout en même temps.

tonial · 2024-10-04T13:32:01Z

tests/www/companies_views/test_card_views.py

+        # If anonymous user, do not show the alert (meaning the context does not contain the job_seeker)
+        response = self.client.get(url)
+        self.assertNotContains(
+            response, f"Vous postulez actuellement pour {job_application.job_seeker.get_full_name()}"


idem dans ce test-ci

tonial · 2024-10-04T13:32:48Z

tests/www/geiq_views/__snapshots__/tests.ambr

@@ -997,7 +997,7 @@

          </ul>

-              <a class="btn btn-secondary btn-block mt-4" href="/company/[PK of Company]/card">Voir la fiche de l'entreprise</a>
+              <a class="btn btn-secondary btn-block mt-4" href="/company/[PK of Company]/card?">Voir la fiche de l'entreprise</a>


c'est normal que le ? apparaisse même quand il n'y a pas de job_seeker ?

tonial · 2024-10-04T13:35:26Z

itou/templates/companies/includes/_company_info.html

@@ -22,7 +22,7 @@ <h3>{{ company.display_name }}</h3>
            <a href="{% url 'apply:job_application_external_transfer_step_1_company_card' job_application_id=job_app_to_transfer company_pk=company.pk %}{% if back_url|default:"" %}?back_url={{ back_url|urlencode }}{% endif %}"
               class="btn btn-secondary btn-block mt-4">Voir la fiche de l'entreprise</a>
        {% else %}
-            <a href="{% url 'companies_views:card' siae_id=company.pk %}{% if back_url|default:"" and not open_in_tab|default:False %}?back_url={{ back_url|urlencode }}{% endif %}"
+            <a href="{% url 'companies_views:card' siae_id=company.pk %}?{% if job_seeker|default:None %}job_seeker={{ job_seeker.public_id }}&{% endif %}{% if back_url|default:"" and not open_in_tab|default:False %}back_url={{ back_url|urlencode }}{% endif %}"


hum... ça donne envie d'ajouter un template tag à qui on passe les arguments et qui les ajoute si la valeur est non nulle.
genre {% add_params job_seeker=job_seeker|default:None, back_url=back_url|default:"" %}

tonial · 2024-10-04T13:39:08Z

tests/www/search/tests.py

+        # Has link to company card with job_seeker public_id
+        company_url_with_job_seeker_id = (
+            f"{guerande_company.get_card_url()}?job_seeker={job_seeker_public_id}"
+            f"&back_url={urlencode_filter(response.wsgi_request.get_full_path())}"


tu peux utiliser urllib.parse.quote à la place de urlencode_filter (c'est la fonction qui est utilisée en dessous)
cf tests/www/apply/test_process_external_transfer.py

tonial · 2024-10-04T13:42:44Z

itou/www/apply/views/submit_views.py

+                    .select_related("jobseeker_profile")
+                    .first()
+                )
+            except (User.DoesNotExist, ValueError, ValidationError):


User.DoesNotExist ne peut pas avoir lieu avec un first().

Le code ressemble beaucoup à celui dans le Mixin du commit précédent.
Il y a peut être moyen de mutualiser dans une fonction séparée ?

tonial · 2024-10-04T13:45:33Z

tests/www/apply/test_submit_from_job_seekers_list.py

+        # Entry point: job seeker details
+        # ----------------------------------------------------------------------
+
+        response = self.client.get(reverse("job_seekers_views:details", kwargs={"public_id": job_seeker.public_id}))


les employeurs n'y ont pas vraiment accès, si ?
Je ne vois pas de lien pour y arriver

EwenKorr added the ajouté Ajouté dans le changelog. label Oct 1, 2024

EwenKorr self-assigned this Oct 1, 2024

EwenKorr force-pushed the ewen/postuler-mes-candidats branch from dba9517 to 1670526 Compare October 1, 2024 15:18

EwenKorr changed the title ~~Candidature : postuler pour un candidat depuis l'espace "Mes candiats"~~ Candidature : postuler pour un candidat depuis l'espace "Mes candidats" Oct 1, 2024

EwenKorr force-pushed the ewen/postuler-mes-candidats branch 4 times, most recently from fa19835 to 5f62cce Compare October 4, 2024 10:21

EwenKorr added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Oct 4, 2024

EwenKorr force-pushed the ewen/postuler-mes-candidats branch from 5f62cce to 9e00177 Compare October 4, 2024 12:05

EwenKorr commented Oct 4, 2024

View reviewed changes

EwenKorr force-pushed the ewen/postuler-mes-candidats branch 2 times, most recently from 5cad6b6 to 17d7693 Compare October 4, 2024 12:21

EwenKorr commented Oct 4, 2024

View reviewed changes

EwenKorr added 3 commits October 4, 2024 14:35

apply: skip create seeker steps when job_seeker GET param is filled

455497e

fixtures: added test job seeker sent by company

621e398

EwenKorr force-pushed the ewen/postuler-mes-candidats branch from 17d7693 to 621e398 Compare October 4, 2024 12:35

EwenKorr commented Oct 4, 2024

View reviewed changes

EwenKorr requested review from tonial, xavfernandez and hellodeloo October 4, 2024 12:46

EwenKorr marked this pull request as ready for review October 4, 2024 12:46

tonial reviewed Oct 4, 2024

View reviewed changes

EwenKorr removed the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Oct 6, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Candidature : postuler pour un candidat depuis l'espace "Mes candidats" #4860

Candidature : postuler pour un candidat depuis l'espace "Mes candidats" #4860

EwenKorr commented Oct 1, 2024 •

edited

Loading

notion-workspace bot commented Oct 1, 2024

github-actions bot commented Oct 4, 2024

EwenKorr Oct 4, 2024 •

edited

Loading

EwenKorr Oct 4, 2024

EwenKorr Oct 4, 2024

EwenKorr Oct 4, 2024

tonial Oct 4, 2024 •

edited

Loading

EwenKorr Oct 4, 2024

EwenKorr Oct 4, 2024

tonial Oct 4, 2024

EwenKorr Oct 4, 2024

EwenKorr Oct 4, 2024

EwenKorr Oct 4, 2024

tonial left a comment

tonial Oct 4, 2024

tonial Oct 4, 2024

tonial Oct 4, 2024

tonial Oct 4, 2024

tonial Oct 4, 2024

tonial Oct 4, 2024

tonial Oct 4, 2024

tonial Oct 4, 2024

tonial Oct 4, 2024

tonial Oct 4, 2024

tonial Oct 4, 2024

		# Create an empty variable to avoid unknown variable template errors in tests
		context["job_seeker"] = None

Candidature : postuler pour un candidat depuis l'espace "Mes candidats" #4860

Are you sure you want to change the base?

Candidature : postuler pour un candidat depuis l'espace "Mes candidats" #4860

Conversation

EwenKorr commented Oct 1, 2024 • edited Loading

🤔 Pourquoi ?

🍰 Comment ?

🚨 À vérifier

🏝️ Comment tester

💻 Captures d'écran

notion-workspace bot commented Oct 1, 2024

github-actions bot commented Oct 4, 2024

EwenKorr Oct 4, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

tonial Oct 4, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

tonial left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

EwenKorr commented Oct 1, 2024 •

edited

Loading

EwenKorr Oct 4, 2024 •

edited

Loading

tonial Oct 4, 2024 •

edited

Loading