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

ETQ que visiteur, je souhaite pouvoir m'inscrire à la news letter #130

Closed
Volubyl opened this issue Nov 15, 2023 · 0 comments
Closed

ETQ que visiteur, je souhaite pouvoir m'inscrire à la news letter #130

Volubyl opened this issue Nov 15, 2023 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@Volubyl
Copy link
Contributor

Volubyl commented Nov 15, 2023

Description du besoin

Actuellement, il n'est plus possible de s'inscrire à la newsletter

Investigation

Quelle est la source du problème ?

Le site web utilise Brevo pour gérer les inscriptions à la newsletter.

Initialement Brevo s'appelait "send in blue" et a changé de nom il y a quelques mois.

Le site web utilise ce SDK pour effectuer le lien avec l'API de Brevo.

Ce SDK est marqué comme déprécié et effectue encore des requêtes vers le nom de domaine send-in-blue

Brevo propose d'utiliser leur nouveau SDK pour remplacer l'ancien.

Seulement, celui-ci ne propose pas de définition TypeScript (Issue : getbrevo/brevo-node#1)

Sans cette définition, l'application ne compile pas.

Erreur :

Error: Could not find a declaration file for module '@getbrevo/brevo'. '/home/volubyl/Repos/mathieujahnich/node_modules/@getbrevo/brevo/src/index.js' implicitly has an 'any' type.
  Try `npm i --save-dev @types/getbrevo__brevo` if it exists or add a new declaration (.d.ts) file containing `declare module '@getbrevo/brevo';` 

Pistes de solution possible

Continuer d'utiliser le SDK déprécié ?

Cela ne me semble pas une bonne idée, car une lib dépréciée est par définition plus maintenue, et donc ouvre la porte à des failles potentielles.

De plus, l'ancien SDK utilise encore le nom de domaine send in blue. Cependant, il n'y a pas d'infos indiquant à quelle date ce nom de domaine sera encore utilisable. Par conséquent, cela risque de poser un problème plus tard.

Se passer du SDK ?

On pourrait directement faire les requêtes avec fetch et se passer du SDK.

C'est une solution possible, mais pas idéale

Utiliser le SDK sans définition

On pourrait forcer Typescript à ignorer cette erreur.

Cela ne semble pas une idée géniale, mais aussi jouable

Dans l'immédiat :

La fonctionnalité ne marche plus, donc il faudrait la retirer du site le temps de trouver la meilleure solution.

Conidérations techniques

@Volubyl Volubyl moved this to 🏁 Prêt à développer in mathieu-jahnich.fr Nov 15, 2023
@Volubyl Volubyl added the bug Something isn't working label Nov 15, 2023
@Volubyl Volubyl self-assigned this Nov 15, 2023
@Volubyl Volubyl closed this as completed Dec 5, 2023
@github-project-automation github-project-automation bot moved this from 🏁 Prêt à développer to En test fonctionnel in mathieu-jahnich.fr Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: En test fonctionnel
Development

No branches or pull requests

1 participant