Fiches salarié: Amélioration de l'affichage des horodatages#6768
Conversation
tonial
force-pushed
the
alaurent/employee_record_summary_notifications_order
branch
2 times, most recently
from
8cc8f45 to
5c842e3
Compare
|
|
||
|
|
||
| def get_asp_batch_file_timestamp(asp_batch_file): | ||
| return datetime.strptime(asp_batch_file[8:22], "%Y%m%d%H%M%S") |
There was a problem hiding this comment.
Vu qu'on utilise le pattern aussi à d'autre endroit, peut-être en faire une constante dans EmployeeRecordBatch ?
Je pense même que la fonction actuelle pourrais être mise là bas car la classe contient déjà les fonctions utilitaires pour le nom de fichier et mon spider sense me dit qu'on va assez vite vouloir la réutiliser pour faire la même chose dans l'admin 😁.
There was a problem hiding this comment.
J'ai aussi ajouté 2 helpers pour créer le nom de fichier et lire la date.
Et j'ai modifié 2 staticmethod qui accèdent à des constants de la class en classmethod
|
|
||
| creations = [ | ||
| ("Mouvement de création", asp_batch_file, get_asp_batch_file_timestamp(asp_batch_file)) | ||
| for asp_batch_file in employee_record.logs.filter(asp_batch_file__isnull=False).values_list( |
There was a problem hiding this comment.
Ici ça me semble préférable de filtrer sur la transition qui correspond à l'envoi que sur la présence du fichier, notre discussion d'hier à relancer mon intuition qu'on devrais sans doute avoir le nom du fichier retour dans les transitions vers l'état retourné par l'ASP (REJECT et PROCESS)
| for asp_batch_file in employee_record.logs.filter(asp_batch_file__isnull=False).values_list( | |
| for asp_batch_file in employee_record.logs.filter(transition=EmployeeRecordTransition.WAIT_FOR_ASP_RESPONSE).values_list( |
| ] | ||
| changes = [ | ||
| ("Mouvement de modification", asp_batch_file, get_asp_batch_file_timestamp(asp_batch_file)) | ||
| for asp_batch_file in employee_record.update_notifications.values_list("asp_batch_file", flat=True) |
There was a problem hiding this comment.
Les NEW n'ont pas encore été envoyées donc on ne veux pas les afficher, et elles n'auront pas de fichier dans tout les cas :
| for asp_batch_file in employee_record.update_notifications.values_list("asp_batch_file", flat=True) | |
| for asp_batch_file in employee_record.update_notifications.exclude(status=Status.NEW).values_list("asp_batch_file", flat=True) |
tonial
force-pushed
the
alaurent/employee_record_summary_notifications_order
branch
from
5c842e3 to
a3519c1
Compare
tonial
force-pushed
the
alaurent/employee_record_summary_notifications_order
branch
2 times, most recently
from
c2b121c to
37ac813
Compare
Display both creation and modification asp batch file names and sort them
They need to access the class.
tonial
force-pushed
the
alaurent/employee_record_summary_notifications_order
branch
from
37ac813 to
19ce58b
Compare
tonial
deleted the
alaurent/employee_record_summary_notifications_order
branch
2 participants
🤔 Pourquoi ?
Pour simplifier les échanges entre l'ASP et les SIAEs
🍰 Comment ?
🚨 À vérifier
🏝️ Comment tester ?
💻 Captures d'écran