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 composant Pixtoggle pour filtrer les campagnes que l'on souhaite voir (PIX-10467) #8921

Merged
merged 1 commit into from
May 16, 2024

Conversation

xav-car
Copy link
Contributor

@xav-car xav-car commented May 15, 2024

🦄 Problème

Nous avons un composant dans le design system pour avoir un toggle, or actuellement sur PixOrga, nous utilisons le toggle custom lorsque 🎵 celui là n'existez pas 🎵 .

🤖 Proposition

Utiliser le composant PixToggle

🌈 Remarques

  • Ce qui me chagrinne dans ce ticket, c'est qu'à mon sens le PixToggle n'est pas accessible en l'état. ( mais quand nous le rendrons accessible sur PixUI. ce sera automatiquement reporté içi)
  • Il faut revoir la traduction du label du PixToggle
  • le sizing du toggle est assez étrange

💯 Pour tester

Aller sur PixOrga et vérifier que l'on peut toujours filtrer sur les campagnes actives/archiver

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

@xav-car xav-car force-pushed the pix-10467/use-pix-ui-toggle-on-campaign branch 2 times, most recently from aab3c32 to 4f8618a Compare May 15, 2024 09:25
@xav-car xav-car marked this pull request as ready for review May 15, 2024 09:52
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.

Fonc KO
Quand on clique sur le toggle et qu'il passe en Archivées , le bouton effacer les filtres s'active. Si on clique dessus, les filtres sont mis à jour (changement d'url, liste des campagnes mises à jour) par contre le toggle ne change pas et reste sur archives 😢

@xav-car xav-car force-pushed the pix-10467/use-pix-ui-toggle-on-campaign branch 3 times, most recently from 5c9518c to 18209fb Compare May 16, 2024 07:25
@lionelB
Copy link
Contributor

lionelB commented May 16, 2024

J'ai oublié de dire dans mon commentaire plus qu'a part le point mentioné, tout fonctionne bien :D 🥂

@lionelB
Copy link
Contributor

lionelB commented May 16, 2024

revue Fonc(Lionel) est OK 🌮
merci @Alexandre-Monney

orga/translations/fr.json Outdated Show resolved Hide resolved
orga/translations/en.json Outdated Show resolved Hide resolved
@lionelB lionelB self-requested a review May 16, 2024 10:05
@lionelB lionelB added Func Review OK PO validated functionally the PR and removed 👀 Func Review Needed 👀 Tech Review Needed labels May 16, 2024
@xav-car xav-car force-pushed the pix-10467/use-pix-ui-toggle-on-campaign branch 2 times, most recently from f7ca67c to 1f2145f Compare May 16, 2024 10:09
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-10467/use-pix-ui-toggle-on-campaign branch from 1f2145f to 0e26c50 Compare May 16, 2024 14:04
@pix-service-auto-merge pix-service-auto-merge merged commit 701407f into dev May 16, 2024
4 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-10467/use-pix-ui-toggle-on-campaign branch May 16, 2024 14:11
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.

None yet

5 participants