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] Monter la version de pix-ui #8793

Merged
merged 14 commits into from
May 3, 2024

Conversation

matthiasferraina
Copy link
Contributor

@matthiasferraina matthiasferraina commented Apr 29, 2024

🦄 Problème

Reboot de la PR

🤖 Proposition

Afin de corriger les regressions d'A11y et d'affichage causées par la montée de version de pix UI (v45.0.0), nous avons fait des modifications de pix-ui (detail ici).

🌈 Remarques

Cette PR embarque donc la version 45.3.1 de pix-ui, sans breaking changes. Cela devrait bien se passer

💯 Pour tester

  • La CI est verte
  • Se balader sur les challenge, et constater que tout est ok sur la RA
  • Se balader partout et constater que tout est ok

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

@La-toile-cosmique La-toile-cosmique marked this pull request as ready for review April 29, 2024 15:12
@La-toile-cosmique La-toile-cosmique requested a review from a team as a code owner April 29, 2024 15:12
@matthiasferraina matthiasferraina force-pushed the renovate/pix-update-pix-ui-to-v45 branch 2 times, most recently from 74b5788 to 3cadc2d Compare April 29, 2024 15:57
@Libouk Libouk force-pushed the renovate/pix-update-pix-ui-to-v45 branch from ac455b1 to 2ec52a5 Compare May 2, 2024 12:31
@matthiasferraina matthiasferraina force-pushed the renovate/pix-update-pix-ui-to-v45 branch 3 times, most recently from 9ced9dd to 07150e0 Compare May 2, 2024 15:10
@matthiasferraina matthiasferraina changed the title [TECH] update pix UI to v45 [TECH] Monter la version de pix-ui May 2, 2024
@matthiasferraina matthiasferraina force-pushed the renovate/pix-update-pix-ui-to-v45 branch 3 times, most recently from 10d4dc8 to eb402e7 Compare May 2, 2024 15:48
@yannbertrand
Copy link
Member

Non blocking : pour info le PixSelect sans placeholder a changé de hauteur.

image

@matthiasferraina matthiasferraina requested a review from a team as a code owner May 3, 2024 07:47
@alexandrecoin
Copy link
Contributor

Fonctionnel ok pour le scope de certif.

Copy link
Contributor

@aurelie-crouillebois aurelie-crouillebois 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 fait un petit tour sur l'app sans voir aucun problème.

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.

J'ai ajouté un correctif à cause d'une régression sur le formulaire de mise à jour de l'adresse email. Régression qui était aussi sur l'autre PR mais qu'on n'avait pas vu 🙈

Correctif testé en RA qui fonctionne bien 🥳
C'est aussi bon pour Accès ✅

@pix-service-auto-merge pix-service-auto-merge force-pushed the renovate/pix-update-pix-ui-to-v45 branch from b67f7f5 to 124f8b3 Compare May 3, 2024 13:54
@pix-service-auto-merge pix-service-auto-merge force-pushed the renovate/pix-update-pix-ui-to-v45 branch from 124f8b3 to 87296f7 Compare May 3, 2024 13:55
@pix-service-auto-merge pix-service-auto-merge merged commit 808baa2 into dev May 3, 2024
6 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the renovate/pix-update-pix-ui-to-v45 branch May 3, 2024 14:01
@yannbertrand
Copy link
Member

Sont embarqués :

  • Fix l'accès à la page des tutos en NL
  • Permettre d'ajouter une opacité sur les couleurs
  • Alléger nos polices d'écriture (-60% en moyenne pour 4 fichiers)
  • Corrige la position du chevron dans le multiselect
  • Améliorer le debug des polices d'écriture
  • Ajout de paramètres sur le composant Pix Collapsible pour afficher un tag

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 Func Review OK PO validated functionally the PR 🚀 Ready to Merge Tech Review OK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants