Candidature: ajout du tri sur les listes de candidatures [GEN-2248]

[xavfernandez]

🤔 Pourquoi ?

Car c'est pratique.

🍰 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

[github-actions]

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

[github-actions]

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

[notion-workspace]

🌿 ETQU je veux pouvoir trier mes tableaux de candidatures par colonne

[EwenKorr]

Ça me paraît bien rôdé comme affaire ! 👍

[EwenKorr]

A priori le annotate() est inutile pour list_for_job_seeker, non ?

[francoisfreitag]

Il est utilisé par l’enum du tri. 857e5e2#diff-a3e650e289e7f2bb2d5f30b585abb74805492b51c6ee98fb828d8553af92f6cfR65

[francoisfreitag]

On pourrait faire une autre enum de tri sans le nom et prénom du candidat, ou désactiver ce critère de tri pour les candidats. Mais ça n’en vaut probablement pas la complexité.

[francoisfreitag]

En relisant les tests, comme on ajoute PK pour stabiliser l’ordre, le candidat peut trier par date d’émission en utilisant la colonne nom 😛

[francoisfreitag]

Au lieu de passer par des attributs, pourquoi ne pas:

diff --git a/itou/static/js/utils.js b/itou/static/js/utils.js
index 436035a8a..6cb575a3e 100644
--- a/itou/static/js/utils.js
+++ b/itou/static/js/utils.js
@@ -214,11 +214,11 @@ htmx.onLoad((target) => {
       if (Array.from(form.elements).some((input) => input.name == checkBoxInputName && input.checked)) {
         document.querySelectorAll(`[${attribute}="hidden"]`).forEach((element) => element.classList.add("d-none"))
         document.querySelectorAll(`[${attribute}="shown"]`).forEach((element) => element.classList.remove("d-none"))
-        document.querySelectorAll(`[${attribute}="disabled"]`).forEach((element) => element.setAttribute("disabled", ""))
+        document.querySelectorAll(`[${attribute}="disabled"]`).forEach((element) => { element.disabled = true; })
       } else {
         document.querySelectorAll(`[${attribute}="shown"]`).forEach((element) => element.classList.add("d-none"))
         document.querySelectorAll(`[${attribute}="hidden"]`).forEach((element) => element.classList.remove("d-none"))
-        document.querySelectorAll(`[${attribute}="disabled"]`).forEach((element) => element.removeAttribute("disabled"))
+        document.querySelectorAll(`[${attribute}="disabled"]`).forEach((element) => { element.disabled = false; })
       }
     })
   })

[francoisfreitag]

Il est utilisé par l’enum du tri. 857e5e2#diff-a3e650e289e7f2bb2d5f30b585abb74805492b51c6ee98fb828d8553af92f6cfR65

[francoisfreitag]

Le code de js: improve data-emplois-elements-visibility-on-selection mechanism est mort, jusqu’à l’introduction du 3ème commit. Je l’aurais fusionné avec le 3ème commit.