-
Notifications
You must be signed in to change notification settings - Fork 24
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
Conversation
There was a problem hiding this 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") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
resend.add_argument("employee_record") | |
resend.add_argument("employee_record", type=int) |
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: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if ready is not None: | |
if ready: |
?
161a3a2
to
6f625ef
Compare
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 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. |
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. |
6f625ef
to
510572b
Compare
J'ai rajouté ma bafouille mais pas dans le |
🤔 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 :
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 :
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.