Skip to content
This repository has been archived by the owner on Nov 6, 2023. It is now read-only.

Posizionamento client GraphQL #8

Closed
bbtgnn opened this issue Nov 1, 2021 · 6 comments · Fixed by #40
Closed

Posizionamento client GraphQL #8

bbtgnn opened this issue Nov 1, 2021 · 6 comments · Fixed by #40

Comments

@bbtgnn
Copy link
Collaborator

bbtgnn commented Nov 1, 2021

L'obiettivo è inizializzare il client GraphQL una volta sola, in questo modo non si ripete ogni volta la procedura di reinizializzazione.

Reference: timhall/svelte-apollo#97

@bbtgnn
Copy link
Collaborator Author

bbtgnn commented Nov 1, 2021

Possibili soluzioni

  • Caricare dentro il layout (ma questa dava qualche problema)
  • Utilizzare uno store (da verificare)

@ZerdoX-x
Copy link

ZerdoX-x commented Dec 7, 2021

Here is the only way how to make apollo (and svelte-apollo) work with svelte: apollographql/apollo-client#8218 (comment)

@bbtgnn
Copy link
Collaborator Author

bbtgnn commented Jan 20, 2022

Per il momento apollo è stato sostituito da graphql-request, molto facile da configurare (con un unico fix da fare indicato qui).

In ogni caso per ogni componente bisogna creare il client, non sono ancora riuscito a farlo funzionare nel layout. Come soluzione per ridurre la ripetizione di codice ho incapsulato tutto il necessario in un'unica funzione.

@bbtgnn
Copy link
Collaborator Author

bbtgnn commented Jan 20, 2022

Closed with #40

@bbtgnn bbtgnn closed this as completed Jan 20, 2022
@bbtgnn bbtgnn linked a pull request Jan 20, 2022 that will close this issue
@bbtgnn
Copy link
Collaborator Author

bbtgnn commented Jan 21, 2022

non sono ancora riuscito a farlo funzionare nel layout

Il problema sembrerebbe essere dato dall'utilizzo di setContext, da indagare. Può sia essere un problema nostro che una limitazione di SvelteKit.

Quindi, ricapitolando: per il momento ogni componente crea un suo client. Se questo diventa troppo impattante a livello di performance si cercherà una soluzione migliore. Per il momento va bene.

@bbtgnn
Copy link
Collaborator Author

bbtgnn commented Mar 11, 2022

Al momento il client graphql non è più utilizzato, in quanto non è possibile effettuare call dalla funzione load di svelte. Servirebbe la versione server del client graphql, ma per il momento va bene utilizzare fetch.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants