Skip to content

[TECH] Supprimer la dépendance à ember-fetch dans Pix Admin. #12018

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

Merged
merged 1 commit into from
Apr 12, 2025

Conversation

HEYGUL
Copy link
Contributor

@HEYGUL HEYGUL commented Apr 10, 2025

🌸 Problème

ember-fetch est déprécié en faveur du fetch natif.
Cette dépendance obsolète nous empêche de migrer vers vite.

Tous les détails sont dans cette RFC : https://rfcs.emberjs.com/id/1065-remove-ember-fetch

🌳 Proposition

On supprime la dépendance ember-fetch et on utilise fetch directement.
Dans les tests, on stubbe désormais window.fetch.

🐝 Remarques

RAS

🤧 Pour tester

CI verte = 😃

@HEYGUL HEYGUL changed the title [TECH] Suppression de ember-fetch dans Pix Admin. [TECH] Supprimer la dépendance à ember-fetch dans Pix Admin. Apr 10, 2025
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@HEYGUL HEYGUL self-assigned this Apr 10, 2025
@HEYGUL HEYGUL added 👀 Tech Review Needed cross-team Toutes les équipes de dev labels Apr 10, 2025
@HEYGUL HEYGUL marked this pull request as ready for review April 10, 2025 09:44
@pix-service-auto-merge pix-service-auto-merge force-pushed the tech-remove-ember-fetch-admin branch from 8071ee8 to f506d99 Compare April 11, 2025 16:50
@pix-service-auto-merge pix-service-auto-merge merged commit 686d049 into dev Apr 12, 2025
10 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the tech-remove-ember-fetch-admin branch April 12, 2025 07:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants