-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feature/searchbydocumentnumberoreppealsnumber #86
Feature/searchbydocumentnumberoreppealsnumber #86
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.
Quelques petits changements a faire/proposés en plus de mes commentaires :
- le bouton filtrer pour les statistiques agrégées se retrouve trop sur la gauche, il y a peut être une marge qui a disparru ? good
- l'affichage de l'agent et du temps est dupliqué lorsque j'ai testé avec 2 statistiques pour le même document traité par le même agent (c'est peut être des cas que tu peux tester en plus dans
fetchDocumentStatistics.spec.ts
)good - l'affichage des dates de traitement et de décision ne semble pas fonctionner good
- Il faudrait idéalement réussir a avoir la même "box" avec l'ombre que pour les statistiques agrégées, avec une marge entre elles si il y en a plusieurs good
- afficher le circuit de relecture - good
- afficher l'importer, cela permet de savoir si c'est un import manuel ou récent good
- afficher la juridiction good
- ne pas afficher les valeurs si elles sont vides/nulles (exemple la chambre dans l'image) good
packages/generic/client/src/pages/Admin/Statistics/StatisticsBox/DocumentStatisticsBox.tsx
Outdated
Show resolved
Hide resolved
packages/generic/client/src/pages/Admin/Statistics/Statistics.tsx
Outdated
Show resolved
Hide resolved
packages/generic/client/src/pages/Admin/Statistics/StatisticsBox/DocumentStatisticsBox.tsx
Outdated
Show resolved
Hide resolved
packages/generic/backend/src/modules/statistic/service/fetchDocumentStatistics.spec.ts
Show resolved
Hide resolved
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.
Je me suis permis de faire le merge de dev + quelques modifs pour clarifier le code ou le style
Issue description :
Get document stats by documentNumber
Describe your changes :
How to test :
Checklist before requesting a review
Désormais l'administrateur peut voir les statistiques dun document, ainsi que l'agent qui l'a traité en passant le document number.