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

[FEATURE] Utiliser PixToast dans Pix Admin (PIX-15328). #10577

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

Conversation

AndreiaPena
Copy link
Member

@AndreiaPena AndreiaPena commented Nov 18, 2024

🍂 Problème

Un nouveau composant de notification à été introduit dans la v48.2.0 de Pix UI.

🌰 Proposition

Remplacer ember-cli-notifications par Pix Toast

🎃 Remarques

  • Montée de version vers la v48.6.2 qui comprend un fix des couleurs de pixIcon
  • Suppression de ember-cli-notifications
  • Les clearAll() sont supprimés car non géré par Pix Toast

🪵 Pour tester

Difficile de lister toutes les pages où se trouve des notifications.
J'ai découpé en plusieurs commits les pages selon les teams qui sont (je pense) owner de celles-ci pour qu'ils puissent tester "leurs bouts".

J'ai forcément du faire des erreurs en les rangeant par team, et j'ai un commit cross team pour les pages dont je n'ai pas réussi à identifier le ou les owners.

1,2,3 , toastez !

@AndreiaPena AndreiaPena added Development in progress cross-team Toutes les équipes de dev labels Nov 18, 2024
@AndreiaPena AndreiaPena self-assigned this Nov 18, 2024
@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 :

@Steph0
Copy link
Contributor

Steph0 commented Nov 20, 2024

Test func, je mets la coche, j'en ai fait plein et rien vu de mal

image

Copy link
Contributor

@Steph0 Steph0 left a comment

Choose a reason for hiding this comment

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

Voir mon autre commentaire

Copy link
Contributor

@lionelB lionelB left a comment

Choose a reason for hiding this comment

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

👍 merci @AndreiaPena
ps : J'ai pas pu tester certains messages d'erreurs

Copy link
Contributor

@er-lim er-lim left a comment

Choose a reason for hiding this comment

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

Lu et testé en RA sur Firefox ✅
Pas pu tester tout les cas notamment les messages d'erreurs.

Merci pour le travail effectué dessus 👏 👏

@AndreiaPena AndreiaPena force-pushed the pix-15328-add-pix-toast-on-pix-admin branch from 078b4ad to a4908f2 Compare November 20, 2024 16:31
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.

6 participants