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

Vérifier l'existence des fiches référencés depuis les contributions lors de la mise à jour des données #2042

Closed
lionelB opened this issue Dec 20, 2019 · 4 comments · Fixed by #2956
Closed

Vérifier l'existence des fiches référencés depuis les contributions lors de la mise à jour des données #2042

lionelB opened this issue Dec 20, 2019 · 4 comments · Fixed by #2956
Assignees
@lionelB
Labels
🛠️ tech 📦 data s: prêt pour dev

Comments

@lionelB
Copy link
Contributor

lionelB commented Dec 20, 2019

Description
Certaine contributions réference des fiche-sp qui ne sont pas indéxées dans ES

Reproduction
Etapes à suivre pour reproduire le bug

  1. Go to http://master.code-du-travail-numerique.dev.factory.social.gouv.fr/contribution/quelle-peut-etre-la-duree-maximale-dun-cdd
  2. See error

Comportement attendu
voir la fiche SP ou a minima gérer l'erreur
@lionelB lionelB added the bug label Dec 20, 2019
@lionelB
Copy link
Contributor Author

lionelB commented Dec 27, 2019

il faut aussi gérer coté frontend le fait qu'on puisse avoir une erreur 404 quand on demande une fiche non référencée

@ClementChapalain ClementChapalain changed the title Fiche SP non référencé depuis les contrivution Fiche SP non référencés depuis les contributions Jan 3, 2020
@ClementChapalain
Copy link
Contributor

ça a été corrigé en changeant le lien pour le nouvel ID.

pour éviter que ça se reproduise, on pourrait imaginer une alerte quand les fiches appelées ne sont pas trouvées ?

@revolunet
Copy link
Member

Toutes les erreurs nous sont remontées éventuellement via sentry mais on pourrait avoir une alerte plus tôt, par exemple au moment où on met à jour les contributions ?

@lionelB
Copy link
Contributor Author

lionelB commented Jan 3, 2020

oui ca serait une bonne vérification !

@lionelB lionelB changed the title Fiche SP non référencés depuis les contributions Vérifier l'existence des fiches référencés depuis les contributions lors de la mise à jour des données Jan 17, 2020
lionelB added a commit that referenced this issue Aug 28, 2020
@lionelB
fix(data): add a check ensure service-public content
554ac75 
ensure content referenced in contribution are not missing
closes #2042
@lionelB lionelB mentioned this issue Aug 28, 2020
Merged
@lionelB lionelB self-assigned this Aug 28, 2020
lionelB added a commit that referenced this issue Sep 17, 2020
@lionelB
fix(data): add a check ensure service-public content (#2956)
f6e9b6c 
ensure content referenced in contribution are not missing
closes #2042
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lionelB lionelB

Labels
🛠️ tech 📦 data s: prêt pour dev
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(data): add a check ensure service-public content SocialGouv/code-du-travail-numerique
3 participants
@revolunet @lionelB @ClementChapalain