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] Utiliser le type error sur les PixNotificationAlert. #10757

Conversation

AndreiaPena
Copy link
Member

@AndreiaPena AndreiaPena commented Dec 9, 2024

🎄 Problème

Le type, alert, a été déprécié sur Pix UI au profit de error. Depuis octobre, le type a été supprimé sur Pix UI mais des composants utilisent encore ce type.

🎁 Proposition

Utiliser le type error.

🧦 Remarques

Quel impact sur les PixNotificationAlert en type alert ?

L'impact est visuel, au lieu d'avoir une notification rouge, on a eu des notifications bleues car le type info (bleu) prend le relais quand le type indiqué est inconnu.


Deux PixMessage dans Pix App utilisent aussi le type alert mais c'est corrigé dans la PR de MAJ de Pix UI actuellement en review.

🎅 Pour tester

Se rendre sur les pages concernées

Pix Orga : Message d'erreur sur le double mire de connexion suite à une invitation dans le formulaire de gauche « Je m’inscris »

Pix Admin :

  • Message d'erreur sur la page de connexion (avec la connexion Google !)
  • Page du référentiel

@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 :

@AndreiaPena AndreiaPena added the cross-team Toutes les équipes de dev label Dec 9, 2024
@AndreiaPena AndreiaPena requested a review from a team December 9, 2024 16:39
@AndreiaPena
Copy link
Member Author

Page du référentiel 🟢
Capture d’écran 2024-12-09 à 18 04 11

Copy link
Contributor

@lego-technix lego-technix left a comment

Choose a reason for hiding this comment

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

✅ Sur la partie PixAdmin authentification Google, j'ai pu reproduire la PixNotificationAlert en bleu, avant la PR, et la PixNotificationAlert en rouge après la PR.

Par contre je n'ai pas encore été capable de reproduire les autres alertes et je n'ai pas pu les tester fonctionnement. Je recommencerai dès que possible. Mais dans tous les cas les changements de cette PR sont logiques et si un des cas avec le changement de type est bon, logiquement les autres devraient l'être !

Copy link
Contributor

@lego-technix lego-technix left a comment

Choose a reason for hiding this comment

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

✅ J'ai réussi à tester fonctionnellement avec succès tous les cas présentés (avec Firefox 🦊)

@pix-service-auto-merge pix-service-auto-merge force-pushed the use-error-instead-of-deprecated-alert-type-on-notification-alert branch from 6d9025a to 04eb336 Compare December 10, 2024 15:54
@pix-service-auto-merge pix-service-auto-merge merged commit 3f65bff into dev Dec 10, 2024
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the use-error-instead-of-deprecated-alert-type-on-notification-alert branch December 10, 2024 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants