Fiches salarié : Modifications sur les filtres [GEN-1816] [GEN-1814]#4974
Fiches salarié : Modifications sur les filtres [GEN-1816] [GEN-1814]#4974
Conversation
rsebille
added
1-recette-jetable
|
🥁 La recette jetable est prête ! 👉 Je veux tester cette PR ! |
rsebille
force-pushed
the
rsebille/er-listing-enhanced
branch
from
6960fb1 to
004dbc9
Compare
xavfernandez
left a comment
xavfernandez
left a comment
There was a problem hiding this comment.
💯 % OK avec les 5 premiers commits mais le dernier m'interpelle 🤔
| }); | ||
| } | ||
| // Clear the job seeker when using the other form (status and order) | ||
| const statusForm = target.querySelector("#employee-record-status-form"); |
There was a problem hiding this comment.
On veut vraiment ce comportement ?
Actuellement ce n'est pas le cas pour les candidatures... Donc je serais plutôt partisan que toutes nos listes se comportent de la même manière avec leurs filtres :-/
There was a problem hiding this comment.
Oui, c'est la demande initiale, je sais plus trop comment ça avais été formulé mais en gros le nom c'est plus une recherche qu'un filtre, actuellement on les traitaient pareils car c'était le plus simple techniquement pour nous, mais que c'est plus naturel pour l'utilisateur de ne pas avoir à choisir les bons statuts quand il recherche par nom vu qu'il sais ce qu'il veux.
Et dans les candidatures (ainsi que PASS IAE) tu peux chercher sans avoir un statut, ce qui avait moins de sens pour les fiches salariés, mais peut-être que c'est ça qu'il faut changer 🤔.
Plutôt d'accord pour avoir le même comportement, je vais demander lequel est la cible et ajuster.
There was a problem hiding this comment.
Ah ça ne me choque pas de chercher une FS sans statut moi: souvent quand je cherche la FS de quelqu'un pour du support je ne sais pas encore quel statut je vais trouver 😅
There was a problem hiding this comment.
C'est la raison de pourquoi une recherche par nom ne prend plus en compte le statut mais dans les pages candidatures et PASS IAE ça le prend en compte, c'est juste que par défaut tu n'as pas de statut sélectionné, mais je pense que le comportement de la page des FS est plutôt la cible que l'on souhaite 🤷.
rsebille
force-pushed
the
rsebille/er-listing-enhanced
branch
from
004dbc9 to
d991f84
Compare
rsebille
force-pushed
the
rsebille/er-listing-enhanced
branch
from
d991f84 to
e043d47
Compare
3 participants
Voir aussi : 2/3 [Améliorer le suivi des FS] Filtre statut fiche salarié : passer des boutons radios aux checkbox
Il reste un retour a propos des "Nouvelle" sans badge mais je me dit qu'on va pas bloquer la PR plus longtemps et le changement sera mineur.
🤔 Pourquoi ?
Car maintenant que les FS hologrammes ne sont plus (#4827) on peux le faire facilement.