Employeur: Paramètre pour fermer les candidatures spontanées [GEN-758] by calummackervoy · Pull Request #5698 · gip-inclusion/les-emplois

calummackervoy · 2025-02-26T16:00:30Z

🤔 Pourquoi ?

Aujourd’hui, il est possible de bloquer toutes les candidatures, ou d’ouvrir et fermer les candidatures pour une fiche de poste en particulier, mais il n’est pas possible de bloquer uniquement les candidatures spontanées.

Or, certaines SIAE ne traitent que les candidatures qui ne sont pas spontanées.
Il faudrait donc pouvoir choisir de bloquer uniquement les candidatures spontanées.

🚨 À vérifier

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

🏝️ Comment tester ?

Avec un compte employeur, manipuler le paramètre par http://localhost:8000/company/job_description_list?page=1

Avec un candidat, tester le moteur de recherche et le parcours de candidature

notion-workspace · 2025-02-26T16:00:34Z

🌿 [Gestion des Fiches de poste] ETQ SIAE je peux choisir d’ouvrir ou fermer la réception des candidatures spontanées

calummackervoy · 2025-02-26T18:30:45Z

On devrait rajouter ce champ à la serialization du SIAE dans l'API ? https://github.com/gip-inclusion/les-emplois/blob/master/tests/api/siae_api/tests.py#L111

calummackervoy · 2025-02-26T18:34:31Z

+                                                </form>
+                                            </td>
+                                            <td></td>
+                                        </tr>


@hellodeloo ce n'est pas la même taille que les autres <tr> sur mon navigateur, il y a une différence de quelques pixels. J'hésitais ajouter un taille fixe pour les <tr>. Qu'est-ce que tu penses ?

Tu peux laisser comme ça. La différence est minime et il est préférable de ne pas bloquer les dimensions

calummackervoy · 2025-02-26T18:36:08Z

+            "Les candidatures spontanées peuvent être ouvertes pendant 90 jours. Une valeur nulle indique que "
+            "les candidatures spontanées sont bloquées."
+        ),
+    )


On a choisis un DateTimeField parce que prochainement ce champs sera mis en False automatiquement chaque 90 jours

calummackervoy · 2025-02-26T18:38:12Z

            ),
        )

+    def test_blocked_application(self, client):


Petit bug corrigé au cours du PR. Je crois que ça vaut le fixe pour éviter un employeur frustré ?

francoisfreitag · 2025-02-27T15:41:28Z


+    @freeze_time("2025-01-01")
+    def test_toggle_spontaneous_applications(self, client, snapshot):
+        response = self._login(client, self.user)


Quelle abstraction bizarre que ce login 🙈. Il n’est pas de ton fait, donc rien à corriger, mais je suis surpris.

francoisfreitag

J’ai un peu abrégé la fin de la revue, plus le temps. Désolé.

francoisfreitag · 2025-02-27T15:49:55Z

+        response = client.get(self.URL, {"city": guerande.slug})
+        assertContains(response, f"<h3>{company.display_name}</h3>")
+        assertContains(response, self.no_spontaneous_applications_str)
+        assertNotContains(response, "Cette structure vous intéresse ?")


Idéalement, on appairerait le texte de cet assertNotContains avec un assertContains pour être sûr que le texte apparaît bien sur la page.

Bonne idée ! ~~J'ai proposé l'ajouter sur une assertion dans un autre test (# Has link to job description)~~

Je ne vois ni la nouvelle assertion ni une variable partagée contenant ce texte ?

francoisfreitag · 2025-02-27T17:13:01Z

+        # Application must not be blocked by the employer at time of access
+        if not self.company.has_member(request.user):
+            if self.company.block_job_applications:
+                messages.error(request, apply_view_constants.ERROR_EMPLOYER_BLOCKING_APPLICATIONS)
+                return HttpResponseRedirect(self.get_redirect_url())
+            if (
+                not len(self.apply_session.get("selected_jobs", []))
+                and not self.company.is_open_to_spontaneous_applications
+            ):
+                messages.error(request, apply_view_constants.ERROR_EMPLOYER_BLOCKING_SPONTANEOUS_APPLICATIONS)
+                return HttpResponseRedirect(self.get_redirect_url())


Je ne suis pas sûr que la complexité en vaille le coup. Ça peut être frustrant pour les candidats et prescripteurs ayant effectué la saisie.
Un processus de candidature ne peut pas durer plus d’une semaine (la session expire), et on ne pourra pas en démarrer un nouveau si l’entreprise bloque les candidatures.
J’aurais laissé les processus de candidature démarrés avant que l’entreprise ne bloque les candidatures se terminer.

Au final, la question est quelle date fait foi : le début du processus de candidature ou sa fin ?

Ça mérite une discussion avec le métier sur le comportement à adopter. On peut leur dire que le fonctionnement actuel est un peu plus simple, mais qu'il peut surprendre les employeurs.

Dans tous les cas, c'est une question intéressante que tu as soulevé 😀

https://gip-inclusion.slack.com/archives/CU8ATF54L/p1741073085846709

francoisfreitag · 2025-02-27T17:14:25Z

Les premiers commits me semblent très propres 👍

github-actions · 2025-03-03T16:57:48Z

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

francoisfreitag

En général ça me semble bien.

Il reste quelques retours de recette à traiter sur la carte.

Idéalement, le blocage du processus de candidature immédiatement à la désactivation d’un poste devrait être sorti dans une autre PR. Ça éviterait de mélanger les retours sur les deux fonctionnalités, et on pourrait intégrer l’une dès qu’elle est prête au lieu d’attendre que les deux ne soient prêtes.

francoisfreitag · 2025-03-10T11:25:46Z

+        response = client.get(self.URL, {"city": guerande.slug})
+        assertContains(response, f"<h3>{company.display_name}</h3>")
+        assertContains(response, self.no_spontaneous_applications_str)
+        assertNotContains(response, "Cette structure vous intéresse ?")


Je ne vois ni la nouvelle assertion ni une variable partagée contenant ce texte ?

francoisfreitag · 2025-03-10T11:30:01Z

On devrait rajouter ce champ à la serialization du SIAE dans l'API ? https://github.com/gip-inclusion/les-emplois/blob/master/tests/api/siae_api/tests.py#L111

Je dirais de ne pas t’embêter, tant qu’on n’a pas de demande.

calummackervoy · 2025-03-10T15:05:39Z

le blocage du processus de candidature immédiatement à la désactivation d’un poste devrait être sorti dans une autre PR

Je ne pense pas que ça fait partie du PR ni la carte, sauf j'ai mal compris ta description ?

francoisfreitag · 2025-03-10T16:20:04Z

le blocage du processus de candidature immédiatement à la désactivation d’un poste devrait être sorti dans une autre PR

Je ne pense pas que ça fait partie du PR ni la carte, sauf j'ai mal compris ta description ?

Je parlais de “Disallow job applications in progress if company blocks mid-process” qui devrait idéalement être sorti dans une autre PR.

calummackervoy · 2025-03-10T16:33:41Z

Je parlais de “Disallow job applications in progress if company blocks mid-process” qui devrait idéalement être sorti dans une autre PR.

OK, c'est fait ! #5748

Via field Company.spontaneous_applications_open_since and control from job_description_list.html

Adds a banner if it's impossible to apply to an enterprise for this reason

…en_since

calummackervoy added the ajouté Ajouté dans le changelog. label Feb 26, 2025

calummackervoy requested a review from francoisfreitag February 26, 2025 16:00

calummackervoy self-assigned this Feb 26, 2025

calummackervoy force-pushed the calum/close-spontaneous-applications branch from 0dcb4cf to 07e7c8e Compare February 26, 2025 18:26

calummackervoy marked this pull request as ready for review February 26, 2025 18:27

calummackervoy requested a review from hellodeloo February 26, 2025 18:32

calummackervoy commented Feb 26, 2025

View reviewed changes

francoisfreitag reviewed Feb 27, 2025

View reviewed changes

hellodeloo approved these changes Feb 27, 2025

View reviewed changes

francoisfreitag reviewed Feb 27, 2025

View reviewed changes

calummackervoy force-pushed the calum/close-spontaneous-applications branch from 07e7c8e to 3e89747 Compare March 3, 2025 16:19

calummackervoy requested a review from francoisfreitag March 3, 2025 16:19

calummackervoy force-pushed the calum/close-spontaneous-applications branch from 3e89747 to b18a5a9 Compare March 3, 2025 16:26

calummackervoy added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Mar 3, 2025

francoisfreitag reviewed Mar 10, 2025

View reviewed changes

Comment thread itou/www/companies_views/views.py Outdated

francoisfreitag reviewed Mar 10, 2025

View reviewed changes

calummackervoy force-pushed the calum/close-spontaneous-applications branch from b18a5a9 to 005e222 Compare March 10, 2025 12:22

calummackervoy force-pushed the calum/close-spontaneous-applications branch from 005e222 to 6bb5397 Compare March 10, 2025 15:21

calummackervoy requested a review from francoisfreitag March 10, 2025 15:37

calummackervoy force-pushed the calum/close-spontaneous-applications branch from 6bb5397 to 8172933 Compare March 10, 2025 16:29

calummackervoy mentioned this pull request Mar 10, 2025

Employeur : empêcher l'envoi de candidatures en cours dès que l'employeur ferme les offres #5748

Merged

2 tasks

xavfernandez approved these changes Mar 10, 2025

View reviewed changes

Comment thread tests/www/search/tests.py Outdated

francoisfreitag approved these changes Mar 11, 2025

View reviewed changes

Comment thread itou/companies/models.py Outdated

Comment thread itou/www/companies_views/views.py Outdated

calummackervoy removed the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Mar 11, 2025

calummackervoy added 4 commits March 11, 2025 13:21

Support Companies closing spontaneous jobseeker applications

8deb3c0

Via field Company.spontaneous_applications_open_since and control from job_description_list.html

Adjust job search results when spontaneous applications are closed

b0f3864

Adds a banner if it's impossible to apply to an enterprise for this reason

ApplicationJobsForm content respects siae.spontaneous_applications_op…

929eabf

…en_since

CompanyCardView adapted for closing of spontaneous applications

3adb7cf

calummackervoy force-pushed the calum/close-spontaneous-applications branch from 8172933 to 3adb7cf Compare March 11, 2025 12:22

calummackervoy added this pull request to the merge queue Mar 11, 2025

Merged via the queue into master with commit 3370247 Mar 11, 2025

calummackervoy deleted the calum/close-spontaneous-applications branch March 11, 2025 12:40

leo-naeka mentioned this pull request Mar 20, 2025

Fiche de poste: Le passage en statut ouvert a un impact sur la date de modification [GEN-113] #5787

Closed

2 tasks

Conversation

calummackervoy commented Feb 26, 2025

🤔 Pourquoi ?

🚨 À vérifier

🏝️ Comment tester ?

Uh oh!

notion-workspace Bot commented Feb 26, 2025

Uh oh!

calummackervoy commented Feb 26, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

francoisfreitag left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

calummackervoy Mar 3, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

francoisfreitag commented Feb 27, 2025

Uh oh!

github-actions Bot commented Mar 3, 2025

Uh oh!

Uh oh!

francoisfreitag left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

francoisfreitag commented Mar 10, 2025

Uh oh!

calummackervoy commented Mar 10, 2025

Uh oh!

francoisfreitag commented Mar 10, 2025

Uh oh!

calummackervoy commented Mar 10, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

calummackervoy Mar 3, 2025 •

edited

Loading