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

migration axios -> redaxios #9

Open
Jean-Baptiste-Lasselle opened this issue Nov 5, 2023 · 1 comment
Open

migration axios -> redaxios #9

Jean-Baptiste-Lasselle opened this issue Nov 5, 2023 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@Jean-Baptiste-Lasselle
Copy link
Contributor

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
Copy link
Contributor Author

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

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants