-
Notifications
You must be signed in to change notification settings - Fork 55
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] Ajouter la nouvelle barre de navigation dans Pix Admin (PIX-16550) #11404
Conversation
Une fois les applications déployées, elles seront accessibles via les liens suivants :
Les variables d'environnement seront accessibles via les liens suivants : |
a7018be
to
808f67c
Compare
f6d2bf3
to
9a6537d
Compare
Hello sur les différents retours :
![]()
![]()
![]() |
8127766
to
c1e0787
Compare
Je fais un message groupé par rapport à tous vos retours (merci pour les feedbacks 🙏 ). J'en ai traité une partie :
@Faraopix C'est corrigé !
@Faraopix Non ça n'a pas de rapport, c'est le composant PixSelect qui n'a pas calculé la bonne taille par rapport aux options. Je ne l'ai pas traité 😛
@AndreiaPena Fait !
@AndreiaPena C'est corrigé 😄
@QuentinChapelain-ui c'est fait, par contre il n'y a pas de FT appliqué. On verra donc cette modification dans la version avec l'ancien menu.
@QuentinChapelain-ui done !
@QuentinChapelain-ui c'est au composant de gérer ça, la gestion devra être faite sur Pix UI |
c1e0787
to
ec3d343
Compare
856b8f1
to
46d9b7b
Compare
with sidebar entry
with pix admin logo
fixup! feat(api): use feature toggle for new sidebar in authenticated hbs
and correct padding for items
46d9b7b
to
a7c5712
Compare
🥞 Problème
La nouvelle barre de navigation n'était pas encore utilisée sur Pix Admin
🥓 Proposition
L'ajouter avec un feature toggle (FT_PIX_ADMIN_NEW_SIDEBAR_ENABLED) le temps que ça soit validé fonctionnellement.
🧃 Remarques
😋 Pour tester
En local
FT_PIX_ADMIN_NEW_SIDEBAR_ENABLED=true
En RA