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.

Sign up for GitHub

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

Jump to bottom

[Action Note] Mettre le statut complété pour les actions de type "NOTE" #1676

Merged
merged 1 commit into from
Sep 4, 2024
Merged

[Action Note] Mettre le statut complété pour les actions de type "NOTE" #1676

merged 1 commit into from
Sep 4, 2024

Conversation

claire2212
Copy link
Collaborator

Related Pull Requests & Issues

  • Tests E2E (Cypress)

@claire2212 claire2212 added the bug Something isn't working label Sep 2, 2024
@claire2212 claire2212 self-assigned this Sep 2, 2024
maximeperraultdev
maximeperraultdev reviewed Sep 4, 2024
View reviewed changes
Copy link
Collaborator

@maximeperraultdev maximeperraultdev left a comment

Choose a reason for hiding this comment

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

Issue: je ne suis pas d'accord avec ce fix. La question est que des notes sont actuellement avec un état TO_COMPLETE alors que quand on enregistre une son état est null et qui devrait le rester.

@claire2212
Copy link
Collaborator Author

claire2212 commented Sep 4, 2024 •
edited
Loading

@maximeperraultdev je pense que ce sont les notes qui étaient créées avant que j'ajoute cette colonne dans la table, comme le statut par défaut est TO_COMPLETE ça me paraît censé. D'ailleurs on le voit bien par rapport aux dates dans le graph de Vincent

@maximeperraultdev
Copy link
Collaborator

@claire2212 Je n'avais pas vu les dates de création de la colonne. Top alors ! c'est réglé 👍

@claire2212 claire2212 merged commit e06fca4 into main Sep 4, 2024
22 checks passed
@claire2212 claire2212 deleted the claire/fix-note-status branch September 4, 2024 13:16
@VincentAntoine
Copy link
Collaborator

OK en inté

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maximeperraultdev maximeperraultdev maximeperraultdev approved these changes

@VincentAntoine VincentAntoine Awaiting requested review from VincentAntoine

Assignees

@claire2212 claire2212

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Statistique][Actions] Les actions de type NOTE sont à l'état TO_COMPLETE
3 participants
@claire2212 @maximeperraultdev @VincentAntoine