Remplacer les appels à helpers/useGraphQL() par hooks/useGraphQL()

[thom4parisot]

Ça éviterait de passer par des useState() à gogo.

Et deux fonctions avec le même nom, une signature identique mais des comportements différents est destabilisant 😅

[ggrossetie]

Après investigation, on ne fait pas la même chose.

Actuellement

helpers/useGraphQL

La méthode useGraphQL dans helpers/graphQL.js permet d'obtenir une fonction permettant d’exécuter requête GraphQL:

const runQuery = useGraphQL()
const handleCreate = useCallback(async () => {
  try {
    const result = await runQuery({
      query: createArticle,
      variables: { user: activeUser._id, title, tags: selectedTagIds },
    })
  } catch (err) {
    setToast({
      text: t('article.create.errorNotification', { errMessage: err }),
      type: 'error',
    })
  }
})

helpers/useGraphQL

La méthode useGraphQL dans hooks/graphQL.js se base sur useSWR et permet de récupérer des données:

const { data: userContactsQueryData, mutate: mutateUserContacts } = useGraphQL(
  { query: getContacts, variables: { userId: activeUserId } },
  {
    revalidateIfStale: false, 
    revalidateOnFocus: false,
    revalidateOnReconnect: false,
  }
)

Proposition

Je propose de transformer helpers/useGraphQL afin de retourner un "client" GraphQL et de renommer useGraphQL en useGraphQLClient :

const graphqlClient = useGraphQLClient()
const result = await graphqlClient.query({
  query: createArticle,
  variables: { user: activeUser._id, title, tags: selectedTagIds },
})

Quand à hooks/useGraphQL, je propose de renommer useGraphQL en useFetchData.
Le "client" GraphQL sera utilisé en tant que fetcher par SWR. Cela permettra de garantir qu'on a le même format d'erreur qu'on utilise SWR ou non (et qu'on execute les requêtes/mutations GraphQL de la même manière).

@thom4parisot est-ce que cela te semble cohérent ?