feat: pouvoir filtrer par actions créées le #1869
Conversation
arnaudambro
had a problem deploying
to
review-auto
GitHub Actions
Error
SocialGroovyBot
temporarily deployed
to
review-feat-created-at-in-new-reports-filter-6aw1ar
| // When status just changed to "todo" we set completedAt to null (since it's not done yet). | ||
| body.completedAt = null; | ||
| } | ||
| if ([DONE, CANCEL].includes(data.status)) { |
There was a problem hiding this comment.
j'ai corrigé ce bug, qui consistait à avoir des completedAt non nuls même avec une action TODO
c'est le même comportement sur les consultations
There was a problem hiding this comment.
donc cette PR fait 2 choses différentes à la fois, tu peux me demander de virer ce correctif si tu veux, j'accepte par avance
There was a problem hiding this comment.
J'ai l'impression que ça a un impact global non négligeable : genre maintenant toute modif sur une action aura un effet sur le completedAt alors qu'avant c'était juste en cas de changement de statut.
Donc comme ça je ne vois pas de gros problème, mais je trouve le changement MAJEUR.
Mais en effet, ça ne semble pas poser de problème.
[EDIT ARNAUD] tu as raison, c'est certes un bug fix mais il nécessite d'autres changements qui l'accompagnent - je retire de cette PR et je fais une autre sur sesan
arnaudambro
force-pushed
the
feat/created-at-in-new-reports-filter
branch
from
7f5cd39
to
3883879
Compare
arnaudambro
had a problem deploying
to
review-auto
GitHub Actions
Error
SocialGroovyBot
temporarily deployed
to
review-feat-created-at-in-new-reports-filter-6aw1ar
arnaudambro
force-pushed
the
feat/created-at-in-new-reports-filter
branch
from
3883879
to
88d6c44
Compare
arnaudambro
temporarily deployed
to
review-auto
GitHub Actions
Inactive
SocialGroovyBot
temporarily deployed
to
review-feat-created-at-in-new-reports-filter-6aw1ar
|
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
There was a problem hiding this comment.
Je ne comprends pas trop les ET et les ou :
J'ai deux actions :
- je coche uniquement "créé pendant cette période" : j'ai les 2 actions
- je coche "à faire" j'ai les 2 actions
- je coche "fait" j'ai aucune action
- je coche "à faire" et "fait" j'ai les deux actions
- je coche "fait" et "créé pendant cette période" je n'ai aucune action.
Du coup parfois les paramètres s'excluent et d'autres fois ils s'ajoutent...
| // When status just changed to "todo" we set completedAt to null (since it's not done yet). | ||
| body.completedAt = null; | ||
| } | ||
| if ([DONE, CANCEL].includes(data.status)) { |
There was a problem hiding this comment.
J'ai l'impression que ça a un impact global non négligeable : genre maintenant toute modif sur une action aura un effet sur le completedAt alors qu'avant c'était juste en cas de changement de statut.
Donc comme ça je ne vois pas de gros problème, mais je trouve le changement MAJEUR.
Mais en effet, ça ne semble pas poser de problème.
[EDIT ARNAUD] tu as raison, c'est certes un bug fix mais il nécessite d'autres changements qui l'accompagnent - je retire de cette PR et je fais une autre sur sesan
arnaudambro
temporarily deployed
to
review-auto
GitHub Actions
Inactive
|
|
🎉 Deployment for commit 3c433f0 : IngressesDocker images
|
|
il y a deux datasets d'actions/consultations qui ne PEUVENT PAS être affichés en même temps (c'est la demande que j'ai comprise en tout cas)
dans chacun de ces datasets, il y a des dans ton cas, tu viens de créer deux actions qui sont
je suis d'accords, c'est pas méga intuitif. |
|
Remplacée : 93cdcd2 |
No description provided.