feat: la configuration applicative est une constante

[ggrossetie]

Permet d'utiliser applicationConfig.graphqlEndpoint uniquement au niveau du fetch.

[netlify]

✅ Deploy Preview for stylo-docs canceled.

Name	Link
🔨 Latest commit	3fbda86
🔍 Latest deploy log	https://app.netlify.com/sites/stylo-docs/deploys/678fa56b0385e800082c73eb

[netlify]

✅ Deploy Preview for stylo-dev ready!

Name	Link
🔨 Latest commit	3fbda86
🔍 Latest deploy log	https://app.netlify.com/sites/stylo-dev/deploys/678fa56b0385e800082c73ed
😎 Deploy Preview	https://deploy-preview-1184--stylo-dev.netlify.app
📱 Preview on mobile	Toggle 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]

@thom4parisot est-ce qu'on fusionne sur master ? ou est-ce que tu vois une "contre-indication" ?

[thom4parisot]

C'est ok ; j'ai juste pas testé les écrans concernés par le changement.

Y'a une raison de ne pas utiliser directement les constantes aux endroits concernés ? (plutôt que de créer un store dont les données ne changent pas)

[ggrossetie]

C'est ok ; j'ai juste pas testé les écrans concernés par le changement.

👍🏻

Y'a une raison de ne pas utiliser directement les constantes aux endroits concernés ? (plutôt que de créer un store dont les données ne changent pas)

C'est uniquement pour centraliser la configuration applicative. Il n'y a de store que le nom, on exporte juste les constantes :

// variables defined statically via vite.config.js
export const applicationConfig = {
  backendEndpoint: __BACKEND_ENDPOINT__,
  graphqlEndpoint: __GRAPHQL_ENDPOINT__,
  exportEndpoint: __EXPORT_ENDPOINT__,
  processEndpoint: __PROCESS_ENDPOINT__,
  pandocExportEndpoint: __PANDOC_EXPORT_ENDPOINT__,
  humanIdRegisterEndpoint: __HUMANID_REGISTER_ENDPOINT__,
  websocketEndpoint: toWebsocketEndpoint(__BACKEND_ENDPOINT__),
}

Tu aurais vu ça comment ? Utiliser directement __BACKEND_ENDPOINT__ et autres dans les fichiers ? Personnellement, je préfère les avoir au même endroit surtout que certaines configurations sont utilisées à plusieurs endroits et il faudrait alors dupliquer les variables __XYZ__