Skip to content

[BUMP] Monter Ember.js en version 5 sur Pix App (PIX-12596) - #8970

Merged
pix-service-auto-merge merged 2 commits into
devfrom
bump-to-ember-5-in-mon-pix
May 21, 2024
Merged

[BUMP] Monter Ember.js en version 5 sur Pix App (PIX-12596)#8970
pix-service-auto-merge merged 2 commits into
devfrom
bump-to-ember-5-in-mon-pix

Conversation

@VincentHardouin

@VincentHardouin VincentHardouin commented May 17, 2024

Copy link
Copy Markdown
Contributor

🦄 Problème

Sur Pix app, nous avons actuellement du retard sur les versions d'Ember.js, la dernière version est la 5.

🤖 Proposition

Passer en version 5 en suivant les conseils de la PR : #8857
Il a aussi fallu forker ember-api-actions pour pouvoir prendre en compte les dépréciations devenues bloquantes.

🌈 Remarques

💯 Pour tester

Faire un tour complet de Pix App

@pix-bot-github

Copy link
Copy Markdown

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

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

@VincentHardouin VincentHardouin self-assigned this May 17, 2024
@yannbertrand yannbertrand changed the title [BUMP] Monter Ember.js en version 5 sur Pix App [BUMP] Monter Ember.js en version 5 sur Pix App (PIX-12596) May 21, 2024
@VincentHardouin
VincentHardouin force-pushed the bump-to-ember-5-in-mon-pix branch 2 times, most recently from 9d8eee7 to 6e2ce8c Compare May 21, 2024 07:54
@VincentHardouin
VincentHardouin marked this pull request as ready for review May 21, 2024 12:30
@VincentHardouin VincentHardouin added 👀 Tech Review Needed cross-team Toutes les équipes de dev dependencies Pull requests that update a dependency file labels May 21, 2024
Comment thread mon-pix/package.json Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

question :
A noter pour plus tard si "non". Est ce que l'on a une volonté de rester sur des ember-api-actions ? ou tout devrait "a terme" être dans les adapter ?

il me semblait qu'on a un moment on était parti pour les supprimer.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai pas d'avis et j'ai pas voulu trancher dans cette PR d'où le fork au lieu de les supprimer, mais oui la question se pose. C'est en effet facilement réalisable avec des adapters ou même des handlers pour aller vers la nouvelle façon de faire.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oui oui, la question c'est pas de le faire now. le fork a du être moins couteux que de tout réecrire 😅 . on a assez d'une migration en V5 😆

@xav-car xav-car left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tour fait sur l'application . passage d'une campagne OK. et plusieurs page me semble tout 🙆‍♂️

reprise de parcours / partage de parcours

@pix-service-auto-merge
pix-service-auto-merge force-pushed the bump-to-ember-5-in-mon-pix branch from 6e2ce8c to f445c60 Compare May 21, 2024 13:14
@pix-service-auto-merge
pix-service-auto-merge deleted the bump-to-ember-5-in-mon-pix branch May 21, 2024 13:32
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 dependencies Pull requests that update a dependency file Tech Review OK

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants