Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fiches salarié : Création d'une commande d'utilitaires #4714

Merged
merged 1 commit into from
Sep 11, 2024

Conversation

rsebille
Copy link
Contributor

🤔 Pourquoi ?

Codifier et versionner certaines actions non disponible ailleurs que depuis www et qui peuvent nous être utile dans le cadre du support.

create

Le problème général est : SIAE à besoin de transférer un PASS pour un SIRET différent de l'actuel
On a actuellement 2 cas différents :

  1. Une SIAE a changée de SIRET en cours d'année, et a besoin de transférer sur son ancien SIRET pour la période avant son changement, la FS actuelle est utilisée pour le nouveau SIRET donc on ne veux pas y toucher
  2. Une SIAE se retrouve découpée en plusieurs structures, et a besoin de transférer sur le nouveau et/ou pour plusieurs SIRET mais une seul candidature existe et elle est sur l'ancien SIRET

L'idée est donc de créer une FS via la commande et de mettre le SIRET désiré afin de la transmettre comme d'habitude.
Solution par cas :

  1. On crée la FS sur la candidature existante et avec l'ancien SIRET, on attend son intégration, on la supprime pour ne pas avoir un doublon de FS et interférer avec la FS portant le SIRET actuel
  2. On crée la FS sur la candidature existante et avec l'ancien SIRET, on attend son intégration, on passe la FS en "Archivée" car le PASS est déjà expiré donc autant garder la trace de l'envoi vu que ce n'est pas gênant

resend

Nous avons un moyen de renvoyer toutes les FS pour une structure mais parfois on veux moins et plus simple ;), la commande aurais pu être refresh mais en général si on veux "rafraîchir" les données dénormalisées c'est pour la renvoyer derrière, c'est foncièrement là comme raccourcis à un désactiver/réactiver, et oui ça pourrais être dans l'admin mais flemme de faire ça maintenant alors qu'on va bientôt attaquer la carte qui ajoutera les transitions et qu'on aura alors une transition "renvoi¨ tout belle.

@rsebille rsebille self-assigned this Sep 10, 2024
@rsebille rsebille added the no-changelog Ne doit pas figurer dans le journal des changements. label Sep 10, 2024
Copy link
Contributor

@francoisfreitag francoisfreitag left a comment

Choose a reason for hiding this comment

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

Merci pour la super description 🙏

create.add_argument("--wet-run", action="store_true")

resend = subparsers.add_parser("resend")
resend.add_argument("employee_record")
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
resend.add_argument("employee_record")
resend.add_argument("employee_record", type=int)

@xavfernandez
Copy link
Contributor

Ce qui serait bien c'est de rajouter une doc quelque part pour dire que ces commandes existent et dans quels cas les utiliser 😅


@staticmethod
def _validate_and_save(obj, ready, wet_run):
if ready is not None:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
if ready is not None:
if ready:

?

@rsebille rsebille force-pushed the rsebille/employee-record-cli branch from 161a3a2 to 6f625ef Compare September 11, 2024 10:12
@rsebille
Copy link
Contributor Author

Ce qui serait bien c'est de rajouter une doc quelque part pour dire que ces commandes existent et dans quels cas les utiliser 😅

Effectivement on pourrais se faire un inventaire des commandes du projet, mais j'ai un peu peur qu'il devienne très vite obsolète 🤔. Ou alors on se dit qu'on documente correctement les commandes et arguments, comme ça l'inventaire c'est ./manage.py --help.

Et pour les deux commandes actuelles, c'est un peu compliqué d'anticiper les cas d'utilisations, ce ne sont sont que des outils à utiliser dans la solution (cf. "Solution par cas") mais pas la solution en elles mêmes comme d'autre commandes.

@francoisfreitag
Copy link
Contributor

Je pense que Xavier suggérait au moins de prendre la description de la PR et de la mettre dans le help text de la commande, histoire qu’on ait une idée des cas où l’utiliser.

@rsebille rsebille force-pushed the rsebille/employee-record-cli branch from 6f625ef to 510572b Compare September 11, 2024 13:16
@rsebille
Copy link
Contributor Author

J'ai rajouté ma bafouille mais pas dans le help car c'était illisible.

@rsebille rsebille enabled auto-merge September 11, 2024 13:17
@rsebille rsebille added this pull request to the merge queue Sep 11, 2024
Merged via the queue into master with commit 873b58d Sep 11, 2024
11 checks passed
@rsebille rsebille deleted the rsebille/employee-record-cli branch September 11, 2024 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog Ne doit pas figurer dans le journal des changements.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants