Skip to content

Fiches salarié : Arrêt des hologrammes (en attente de création) [GEN-1812]#4827

Merged
rsebille merged 5 commits intomasterfrom
rsebille/er-remove-hologram
Oct 21, 2024
Merged

Fiches salarié : Arrêt des hologrammes (en attente de création) [GEN-1812]#4827
rsebille merged 5 commits intomasterfrom
rsebille/er-remove-hologram

Conversation

@rsebille
Copy link
Contributor

🤔 Pourquoi ?

Simplification technique pour améliorer ensuite l'UI/UX

🍰 Comment ?

Je voulais découper le commit principal en 2, 1 partie qui ne touche qu'à la liste et une autre qui supprime tout ce qui était eligible_as_employee_record() dorénavant plus utilisé mais les 2 sont connectés via le message "En attente de création" de l'admin donc j'ai tout regroupé.

J'ai supprimé le bandeau d'alerte général "Une action de votre part est nécessaire" et tout ce qui était need_manual_regularization qui ne concerne désormais que les régularisations de NIR, mais avec la réduction drastique du nombre de FS je me dit qu'on a plus trop besoin de prévenir l'utilisateur car il le verra soit dans la liste soit en allant sur la FS.

Et pour simplifier j'ai "déplacé" le cas "Régulariser le n° de sécurité sociale" pour l'avoir sur tout les statuts au lieu de juste "Nouvelle", c'est le plus simple et ça ne me semble pas déconnant de l'avoir si jamais il faut faire un renvoi par exemple.

🏝️ Comment tester

  1. Accepter une candidature, prolonger ou suspendre un PASS
  2. Aucune FS n'est présente dans "Nouvelle" (liste et badge)
  3. Pour le cas "candidature acceptée" : La FS peux être créée via le bouton "Créer une fiche salarié"

@rsebille rsebille added the supprimé Supprimé dans le changelog. label Sep 26, 2024
@rsebille rsebille self-assigned this Sep 26, 2024
@notion-workspace
Copy link

Copy link
Contributor

@xavfernandez xavfernandez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Techniquement cela m'a l'air bien (et plus propre forcément 😅 ).
D'un point de vue métier, j'ai un peu peur que les employeurs pensent moins à créer leur FS dès l'acceptation 😬

Copy link
Contributor Author

@rsebille rsebille left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Techniquement cela m'a l'air bien (et plus propre forcément 😅 ). D'un point de vue métier, j'ai un peu peur que les employeurs pensent moins à créer leur FS dès l'acceptation 😬

Je ne pense pas que ça sera un problème, ils ne le font déjà pas tous systématiquement donc leur workflow ne va pas trop changer parce qu'on propose plus, surtout qu'ils ont pas le choix pour avoir les choses cotés Extranet IAE.
Et le fait de ne pas "presser" aura peut-être un effet sur l'importance ressenti et éventuellement ça évitera des histoires de "j'ai embauché la personne, j'ai envoyé la FS, mais la personne n'est jamais venu, etc".

La partie qui me fait le plus peur c'est de ne plus proposer une FS suite à une prolongation/suspension, car certes les employeurs en laisse trainer plein dans leur tableau de bord mais ils doivent en remplir aussi de temps en temps :).

@rsebille rsebille force-pushed the rsebille/er-remove-hologram branch from 2d3c69b to a8e9a87 Compare October 3, 2024 14:04
@rsebille rsebille added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Oct 15, 2024
@rsebille rsebille force-pushed the rsebille/er-remove-hologram branch from a8e9a87 to bc32674 Compare October 15, 2024 09:03
@rsebille rsebille added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC and removed 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC labels Oct 15, 2024
@github-actions
Copy link

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

@rsebille rsebille force-pushed the rsebille/er-remove-hologram branch from bc32674 to 6a8a538 Compare October 21, 2024 09:32
It's not really elegant but otherwise if an employer try to create an
employee record for a job seeker in that state it will be stuck with the
vanilla error message.
@rsebille rsebille force-pushed the rsebille/er-remove-hologram branch from 6a8a538 to 30cd6c3 Compare October 21, 2024 10:07
@rsebille rsebille added this pull request to the merge queue Oct 21, 2024
Merged via the queue into master with commit 8de590a Oct 21, 2024
@rsebille rsebille deleted the rsebille/er-remove-hologram branch October 21, 2024 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC supprimé Supprimé dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants