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

chore: supprime useMutation et clarifie les fonctions #1304

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

ggrossetie
Copy link
Collaborator

  • Renomme useGraphQL en useFetchData
  • Introduit une fonction useMutateData pour mettre à jour les données pour un scope (query + variables)

Copy link

netlify bot commented Feb 26, 2025

Deploy Preview for stylo-docs canceled.

Name Link
🔨 Latest commit ed44e6d
🔍 Latest deploy log https://app.netlify.com/sites/stylo-docs/deploys/67c847c1f065280008b792d9

Copy link

netlify bot commented Feb 26, 2025

Deploy Preview for stylo-dev ready!

Name Link
🔨 Latest commit ed44e6d
🔍 Latest deploy log https://app.netlify.com/sites/stylo-dev/deploys/67c847c10c13a30008d3068a
😎 Deploy Preview https://deploy-preview-1304--stylo-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@ggrossetie
Copy link
Collaborator Author

Je pense qu'il y a quelques endroits où on va pouvoir remplacer les fonctions de type onSomething{Updated|Created|Deleted} avec un mutate scoped. Je préfère le faire dans un second temps pour ne pas introduire trop de changements 😄

Copy link
Member

@thom4parisot thom4parisot left a comment

Choose a reason for hiding this comment

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

Je m'arrête là parce que j'ai tenté 3 actions et y'avait 3 erreurs.

Je veux bien tester mais pas faire tout le boulot.

@ggrossetie
Copy link
Collaborator Author

Je m'arrête là parce que j'ai tenté 3 actions et y'avait 3 erreurs.
Je veux bien tester mais pas faire tout le boulot.

Désolé, il manquait des bouts 😬
Je vais essayer d'ajouter des tests automatisés pour attraper une partie des erreurs.

@ggrossetie ggrossetie requested a review from thom4parisot March 4, 2025 15:30
@ggrossetie ggrossetie force-pushed the fix-1171-remove-deprecated branch from 410e0e7 to ed44e6d Compare March 5, 2025 12:46
@ggrossetie
Copy link
Collaborator Author

@thom4parisot c'est rebase sur master et j'ai ajouté des tests, ça devrait être mieux 😄

Pour les tests, ça reste superficiel car on mock fetch donc on valide uniquement qu'une requête est envoyée mais la requête GraphQL peut être incomplète. Cela vaut peut être le coup d'avoir un serveur GraphQL avec des mocks : https://www.apollographql.com/docs/apollo-server/testing/mocking ? Sinon des tests e2e avec https://playwright.dev/ ... ? à discuter 💬

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.

2 participants