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

[TECH] Correction des deprecations EmberJs sur Pix Admin #11800

Open
wants to merge 8 commits into
base: dev
Choose a base branch
from

Conversation

bpetetot
Copy link
Contributor

@bpetetot bpetetot commented Mar 24, 2025

🌸 Problème

Certaines fonctionnalités EmberJS sont dépréciées et doivent être traitées pour les mises à jour futures de emberjs.

🌳 Proposition

Cette PR se concentre sur https://deprecations.emberjs.com/id/deprecate-array-prototype-extensions/

Difficulté d'appliquer la modification des fonctions pushObject, removeObject, clear, removeAtsur les fichiers suivants:

  • Problème de mutation sur des arguments de composant de type tableau :
    • admin/app/components/target-profiles/badge-form/criteria.gjs
    • admin/app/components/organizations/all-tags.gjs
    • admin/app/controllers/authenticated/complementary-certifications/complementary-certification/attach-target-profile/new.js
    • admin/app/components/complementary-certifications/attach-badges/index.gjs
  • Fichier complexe et difficilement modifiable tel quel :
    • admin/app/components/target-profiles/stages.gjs

🐝 Remarques

🤧 Pour tester

@bpetetot bpetetot self-assigned this Mar 24, 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 :

@bpetetot
Copy link
Contributor Author

bpetetot commented Mar 24, 2025

@VincentHardouin Merci pour l'approuve mais ce n'est pas terminé 😅 Je fais un commit par fonction modifiée et tourner la CI sur les commits pour voir si ça pète rien. 😄

@bpetetot bpetetot requested a review from a team as a code owner March 24, 2025 08:57
@bpetetot bpetetot force-pushed the fix-pix-admin-ember-deprecation branch 2 times, most recently from 4d4b1ad to 0e84540 Compare March 24, 2025 09:38
@bpetetot bpetetot requested a review from a team as a code owner March 24, 2025 09:38
@VincentHardouin VincentHardouin self-requested a review March 24, 2025 09:47
@bpetetot bpetetot force-pushed the fix-pix-admin-ember-deprecation branch 4 times, most recently from b0fe35d to 349e23e Compare March 24, 2025 14:19
@bpetetot bpetetot force-pushed the fix-pix-admin-ember-deprecation branch from 349e23e to 018ea28 Compare March 24, 2025 14:30
@bpetetot bpetetot force-pushed the fix-pix-admin-ember-deprecation branch from 018ea28 to 92a27de Compare March 24, 2025 15:32
@bpetetot bpetetot added the cross-team Toutes les équipes de dev label Mar 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cross-team Toutes les équipes de dev
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants