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.

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

[TRA-14208] Faire remonter BSD dans dashboard à la création de demande de révision #3315

Merged
merged 2 commits into from
May 28, 2024

Conversation

silto
Copy link
Collaborator

@silto silto commented May 11, 2024

Modification

Afin de faire remonter le bordereau en cas de demande de révision, on va le "touch" en mettant à jour sa propriété updatedAt (avec un select minimal puisque le résultat ne nous intéresse pas).

Webhook?

Cette demande originale amène possiblement à un autre ticket, où est faite la demande d'avoir un appel webhook en cas de revisionRequest (voir 2e lien de ticket). C'est déjà le cas avec un appel UPDATED concernant le bordereau (puisqu'un reindex est fait dessus, ce qui déclenche un webhook). Celà dit il est envisageable de rajouter un nouveau type d'évènement, ou bien des détails sur l'update qui a eu lieu en modifiant le type d'objet, etc. Celà dit ce sont des breaking changes, donc à discuter (question abordée avec @GaelFerrand )

--> A discuter et spec (nouveau type d'évènement, ajout de qq infos sur les updates,...)
--> @GaelFerrand Le webhook est bian appelé avec un event UPDATE dans le cas create/cancel/accept des revisionRequest

Date de modification

Comme remonté à @providenz les dates affichées dans "Modifié le" sur les BsdCard ne correspondent pas au updatedAt des BSD (utilise le stateSummary pour l'affichage), ce qui amène à un classement qui peut sembler aléatoire. Il faudrait peut-être harmoniser l'affichage et l'ordre des BSD, soit en utilisant updatedAt pour l'affichage, soit en essayant de voir pourquoi la date remontée dans stateSummary ne correspond pas (je n'ai pas encore cherché).

--> A discuter et spec (la date du stateSummary correspond bien à qqch de concret, mais mismatch avec le classement des bordereaux, donc possiblement changer le wording "Modifié le" ou ajouter updatedAt qq part)

Demo

Demo.revision.bsd.updatedat.mov

  • Mettre à jour la documentation
  • Mettre à jour le change log
  • Documenter les manipulations à faire lors de la mise en production (sur le ticket Favro de release)
  • S'assurer que la numérotation des nouvelles migrations est bien cohérente
  • Informer le data engineer de tout changement de schéma DB

@silto silto changed the title Faire remonter BSD dans dashboard à la création de demande de révision [TRA-14208] Faire remonter BSD dans dashboard à la création de demande de révision May 11, 2024
Copy link

sonarcloud bot commented May 28, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@silto silto merged commit 045cdff into dev May 28, 2024
17 checks passed
@silto silto deleted the tra-14208 branch May 28, 2024 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants