3forges / poc-preact-rtk-flowbite

0 stars 1 forks source link

migration axios -> redaxios #9

Open Jean-Baptiste-Lasselle opened 9 months ago

Jean-Baptiste-Lasselle commented 9 months ago

Hm, en train de finir mes infusions, j'ai regardé cette vidéo sur "ne plus utiliser axios"

Tu avais raison, @BorisTherin il y avait quelque chose, et on va du coup changer un truc dans pesto.

Donc, le fait est que axios utilise , "en dessous", XML Http request, alors qu'il faut utiliser fetch à la place.

Attention, cela n'est vrai que dans le browser, j'ai déjà utilisé axios côté serveur, et là il est certain qu'axios n'utilise pas un objet du browser comme XML Http request : là il y a une petite étude à mener pour éclaircir la situation lorsque l'on est pas dans le browser. Cette étude, nous la mettons dans l'escarcelle, on reviendra dessus après la première release officielle de pesto.

Cette nouvelle situation, qui nous amène à ce changement, existe depuis 2022 : c'est très récent, et merci d e 'avori dite totu de suite, grâce à toi, on va faire ce changement très tôt, quand encore peu sont conscient de cela.

Il y a Ky, comme tu l'as dit. Il y a aussi redaxios

On va utiliser redaxios, pour 2 raisons:

image

avec redaxios on a juste à changer les import et boum c fini c bon magique

Point technique Important!

Pour faire cette migration, que tu vas faire @BorisTherin , il faut d'abord que je finisse le travail RTK Queries sur ma branche jbl/clean/up : que je l'ai mergée dans master.

Jean-Baptiste-Lasselle commented 9 months ago

Ps: pour relativiser le point de vue exposé dans la vidéo youtube, voici une page de la doc de create-react-app :

image