Employeur : prescrire une PMSMP depuis les Emplois [GEN-2269] by EwenKorr · Pull Request #6334 · gip-inclusion/les-emplois

EwenKorr · 2025-06-13T14:48:46Z

🤔 Pourquoi ?

Intégrer un encart pour informer l’employeur qu’il peut proposer une PMSMP au candidat.

Dans la page de détail d’une candidature ; ne concerne que les candidatures adressées par un prescripteur habilité

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

Mettre à jour le CHANGELOG_breaking_changes.md ?
Ajouter l'étiquette « Bug » ?

🏝️ 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 · 2025-06-13T14:48:51Z

🌿 Prescrire une PMSMP depuis les Emplois

github-actions · 2025-06-16T08:02:53Z

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

EwenKorr · 2025-06-16T10:10:35Z

                                        {% endif %}
                                    {% endif %}
                                {% endwith %}
+                                {% include "apply/includes/pmsmp_box.html" with link_if_pmsmp=link_if_pmsmp %}


link_if_pmsmp n'est défini que dans views.process_views.details_for_company.
Donc il n'est pas défini dans les vues qui utilisent le template apply/process_base.html.

Je me rappelle plus, on n'est pas censé avoir une erreur dans les tests (unknown_variable_template_error) ?

Bien vu, je pense que Django 5.2 a fait sauter la vérif 😱

C'est pour ça que j'ai pu impunément insérer mon premier commit alors !

hellodeloo · 2025-06-16T12:47:36Z

+        <div class="c-box__header--immersion-facile p-3 p-lg-4">
+            <span class="h4 m-0">Immersion facilitée</span>
+        </div>
+        <div class="p-3 p-lg-4">


Moi je proposerais bien d'utilser <abbr>. Il est fait pour ça.
Genre ça à la place

<div class="p-3 p-lg-4"> <p> La <abbr title="Période de mise en situation en milieu professionnel">PMSMP</abbr> permet à un candidat de découvrir un métier ou valider un projet dans votre structure, sans obligation d’embauche. En cliquant sur « <strong>Proposer une PMSMP</strong> », vous accédez à un formulaire prérempli sur <i>Immersion Facilitée</i>. </p> <a href="{{ link_if_pmsmp }}" class="btn-link has-external-link" rel="noopener" target="_blank" aria-label="Proposer une PMSMP (ouverture dans un nouvel onglet)"> Proposer une PMSMP </a> </div>

Ça me va très bien !

tonial

Attention, la recette jetable envoi sur https://immersion-facile.beta.gouv.fr/demande-immersion?agencyDepartment=13&agencyKind=pole-emploi&siret=35217815600027&skipIntro=true&acquisitionCampaign=emplois&mtm_kwd=candidature

Est-ce qu'on ne veut pas masquer l'encart sous certaines conditions ? ou desactiver le lien quand on n'est pas sur la prod (en indiquant avec un tooltip que ce n'est disponible qu'en prod) ?

EwenKorr · 2025-06-17T08:37:32Z

Est-ce qu'on ne veut pas masquer l'encart sous certaines conditions ? ou desactiver le lien quand on n'est pas sur la prod (en indiquant avec un tooltip que ce n'est disponible qu'en prod) ?

Je ne sais pas à quel point c'est problématique, mais ce qu'on peut faire c'est surcharger la variable IMMERSION_FACILE_SITE_URL par https://staging.immersion-facile.beta.gouv.fr quand on n'est pas en prod ?

leo-naeka · 2025-06-17T08:46:52Z

Est-ce qu'on ne veut pas masquer l'encart sous certaines conditions ? ou desactiver le lien quand on n'est pas sur la prod (en indiquant avec un tooltip que ce n'est disponible qu'en prod) ?

Je ne sais pas à quel point c'est problématique, mais ce qu'on peut faire c'est surcharger la variable IMMERSION_FACILE_SITE_URL par https://staging.immersion-facile.beta.gouv.fr quand on n'est pas en prod ?

Le risque c'est de faire une vraie demande d'immersion depuis nos déploiements de démo / recette / etc. (bon, à ce stade il y a quand même beaucoup d'items à remplir, mais avec un peu d'inattention c'est vite plié…

Éventuellement baser ces constates sur des env vars dans une autre PR (en mettant celles de démo/staging par défaut) ?

les-emplois/itou/utils/constants.py

Lines 7 to 9 in f0caf5e

    
           PILOTAGE_SITE_URL = "https://pilotage.inclusion.beta.gouv.fr" 
        
           EMPLOIS_SITE_URL = "https://emplois.inclusion.beta.gouv.fr" 
        
           IMMERSION_FACILE_SITE_URL = "https://immersion-facile.beta.gouv.fr"

tonial · 2025-06-17T08:47:20Z

Dans ce cas là, je le mettrai en setting : la valeur par défaut c'est staging, et dans les settings de prod tu mets la vrai urls
Ou alors il n'y a pas de valeur par défaut et on indique systématiquement une url dans tous les settings, mais ça nous force en local à le renseigner.
Je préfère la 1ere option

EwenKorr · 2025-06-17T09:20:38Z

Dans ce cas là, je le mettrai en setting : la valeur par défaut c'est staging, et dans les settings de prod tu mets la vrai urls

C'est ce que j'avais en tête également. :)
#6361

tonial

👍

Only specific employers see it, on job applications sent by authorized prescribers.

EwenKorr added the ajouté Ajouté dans le changelog. label Jun 13, 2025

EwenKorr self-assigned this Jun 13, 2025

EwenKorr added the ajouté Ajouté dans le changelog. label Jun 13, 2025

EwenKorr changed the title ~~Candidature : prescrire une PMSMP depuis les Emplois [GEN-2269]~~ Employeur : prescrire une PMSMP depuis les Emplois [GEN-2269] Jun 13, 2025

EwenKorr marked this pull request as draft June 13, 2025 14:49

EwenKorr force-pushed the ewen/pmsmp branch from 85b9ffe to 36d6277 Compare June 13, 2025 14:56

EwenKorr added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Jun 16, 2025

EwenKorr force-pushed the ewen/pmsmp branch 2 times, most recently from bbaf570 to 21207c1 Compare June 16, 2025 09:51

EwenKorr marked this pull request as ready for review June 16, 2025 10:03

EwenKorr requested review from hellodeloo, leo-naeka and xavfernandez June 16, 2025 10:04

EwenKorr commented Jun 16, 2025

View reviewed changes

hellodeloo reviewed Jun 16, 2025

View reviewed changes

EwenKorr force-pushed the ewen/pmsmp branch 5 times, most recently from 242b616 to c621219 Compare June 16, 2025 15:16

leo-naeka reviewed Jun 16, 2025

View reviewed changes

EwenKorr force-pushed the ewen/pmsmp branch 3 times, most recently from 923c778 to 7427a56 Compare June 17, 2025 08:16

tonial reviewed Jun 17, 2025

View reviewed changes

Comment thread itou/utils/immersion_facile.py Outdated

EwenKorr force-pushed the ewen/pmsmp branch from 7427a56 to fa011f6 Compare June 17, 2025 08:55

EwenKorr mentioned this pull request Jun 17, 2025

Settings : définir IMMERSION_FACILE_SITE_URL par une variable d'environnement #6361

Merged

tonial approved these changes Jun 17, 2025

View reviewed changes

xavfernandez approved these changes Jun 17, 2025

View reviewed changes

EwenKorr force-pushed the ewen/pmsmp branch from fa011f6 to 4f8cdf7 Compare June 17, 2025 09:42

EwenKorr added 2 commits June 17, 2025 11:59

utils.immersion_facile: add get_pmsmp_url helper

c428562

apply: add a box to apply to a PMSMP on Immersion Facilitée

881bf6e

Only specific employers see it, on job applications sent by authorized prescribers.

EwenKorr force-pushed the ewen/pmsmp branch from 4f8cdf7 to 881bf6e Compare June 17, 2025 09:59

leo-naeka approved these changes Jun 17, 2025

View reviewed changes

EwenKorr added this pull request to the merge queue Jun 17, 2025

Merged via the queue into master with commit 0c7af37 Jun 17, 2025

EwenKorr deleted the ewen/pmsmp branch June 17, 2025 15:11

Conversation

EwenKorr commented Jun 13, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🤔 Pourquoi ?

🍰 Comment ?

🚨 À vérifier

🏝️ Comment tester ?

💻 Captures d'écran

Uh oh!

notion-workspace Bot commented Jun 13, 2025

Uh oh!

github-actions Bot commented Jun 16, 2025

Uh oh!

EwenKorr Jun 16, 2025

Choose a reason for hiding this comment

Uh oh!

xavfernandez Jun 16, 2025

Choose a reason for hiding this comment

Uh oh!

EwenKorr Jun 16, 2025

Choose a reason for hiding this comment

Uh oh!

hellodeloo Jun 16, 2025

Choose a reason for hiding this comment

Uh oh!

EwenKorr Jun 16, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

tonial left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

EwenKorr commented Jun 17, 2025

Uh oh!

leo-naeka commented Jun 17, 2025

Uh oh!

tonial commented Jun 17, 2025

Uh oh!

EwenKorr commented Jun 17, 2025

Uh oh!

tonial left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

EwenKorr commented Jun 13, 2025 •

edited

Loading