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

fix(contributions): correction des contributions avec une fiche SP #1501

Merged
merged 1 commit into from
Nov 5, 2024

Conversation

m-maillot
Copy link
Contributor

@m-maillot m-maillot commented Nov 5, 2024

closes #1502

@@ -45,7 +45,7 @@ export const getContributionAnswerById = gql`
cdtn_references {
cdtn_id
}
content_fiche_sp: document {
content_fiche_sp: document_fiche_sp {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Le problème vient de cette relation. On venait mettre le document dans content_fiche_sp sauf que le document c'est le document de la contrib dans la table documents et non de la fiche SP.
De ce fait, au query, on récupère le cdtn_id de la contrib au lieu de la fiche SP.

Comment on lines +18 to +20
- name: document_fiche_sp
using:
foreign_key_constraint_on: content_service_public_cdtn_id
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Il manquait la relation vers la fiche SP dans hasura.

Copy link

sonarqubecloud bot commented Nov 5, 2024

Copy link

github-actions bot commented Nov 5, 2024

🎉 Deployment for commit 1cef976 :

Ingresses
Docker images
  • 📦 docker pull harbor.fabrique.social.gouv.fr/cdtn/cdtn-admin/export:sha-1cef976bc223367444ec7069f82b56534b2445f5
  • 📦 docker pull harbor.fabrique.social.gouv.fr/cdtn/cdtn-admin/frontend:sha-1cef976bc223367444ec7069f82b56534b2445f5
  • 📦 docker pull harbor.fabrique.social.gouv.fr/cdtn/cdtn-admin/hasura:sha-1cef976bc223367444ec7069f82b56534b2445f5
Debug

@carolineBda carolineBda merged commit 908d73c into master Nov 5, 2024
25 checks passed
@carolineBda carolineBda deleted the fix/wrong-sp branch November 5, 2024 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Erreur dans la mise à jour des données en prod
4 participants