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 PixTable dans les tableaux d'équipe et de finalisation sur Pix Certif (PIX-16788). #11545

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

Conversation

AndreiaPena
Copy link
Member

@AndreiaPena AndreiaPena commented Feb 28, 2025

🥞 Problème

Nous avons un composant, PixTable, pour remplacer nos anciens tableaux. Il n'est pas utilisé sur Pix Certif.

🥓 Proposition

Utiliser PixTable

🧃 Remarques

Première PR sur les tableaux de Pix Certif
#11521

😋 Pour tester

Pour @1024pix/team-acces

  • Se connecter avec [email protected]
  • Aller dans le menu Équipe
  • New : la modification de rôle ne se fait plus dans le tableau même mais dans une modale
  • Le wording a été fait avec @pierrepougetpix, à faire review par @MathGuich :D
  • Faire de la non reg avec un autre environnement pour :
  • Plus de menu avec des actions, c'est désormais des boutons avec icônes, validées par @pierrepougetpix
  • La modification d'un rôle => on a corrigé l'effet de bord qui faisait apparaître les boutons lorsqu'on changeait le rôle d'un camarade de member à admin
  • Vérifier que la modification de rôle s'effectue bien et que le membre ait accès à des boutons (ou plus accès) selon son rôle

L'utilisateur connecté ET membre ne voit rien en terme de bouton d'action, il ne voit même pas la colonne

L'utilisateur connecté ET administrateur voit les boutons, il peut modifier et supprimer un autre membre. Pour lui même il peut se supprimer de l'espace UNIQUEMENT s'il y a un autre administrateur

  • La colonne Référent a été placé avant la colonne actions
Capture d’écran 2025-03-06 à 10 41 00

Tableaux lors de la finalisation de session

Capture d’écran 2025-03-06 à 11 01 52

@AndreiaPena AndreiaPena self-assigned this Feb 28, 2025
@AndreiaPena AndreiaPena changed the base branch from dev to pix-15794-add-pix-table-on-certif February 28, 2025 16:10
@AndreiaPena AndreiaPena force-pushed the pix-16788-add-pix-table-on-team-table branch from d0811e8 to 446fa30 Compare February 28, 2025 16:11
@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 force-pushed the pix-15794-add-pix-table-on-certif branch from 92bbbe9 to 175d0a8 Compare February 28, 2025 16:23
@AndreiaPena AndreiaPena force-pushed the pix-16788-add-pix-table-on-team-table branch 3 times, most recently from 25c5e45 to c784828 Compare March 3, 2025 13:30
@AndreiaPena AndreiaPena marked this pull request as ready for review March 3, 2025 13:30
@AndreiaPena AndreiaPena added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally ⚠️ PR Inheritance This PR inherits a first-to-merge PR and will need a rebase 👀 Design Review Needed and removed Development in progress 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally labels Mar 3, 2025
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-15794-add-pix-table-on-certif branch from d13efe7 to c130073 Compare March 3, 2025 15:34
Base automatically changed from pix-15794-add-pix-table-on-certif to dev March 3, 2025 15:40
@AndreiaPena AndreiaPena added Development in progress and removed ⚠️ PR Inheritance This PR inherits a first-to-merge PR and will need a rebase labels Mar 3, 2025
@AndreiaPena AndreiaPena force-pushed the pix-16788-add-pix-table-on-team-table branch from c784828 to a3860c1 Compare March 5, 2025 17:00
@AndreiaPena AndreiaPena changed the title [FEATURE] Utiliser PixTable dans le tableau équipe sur Pix Certif (PIX-16788). [FEATURE] Utiliser PixTable dans les tableaux d'équipe et de finalisation sur Pix Certif (PIX-16788). Mar 5, 2025
@AndreiaPena AndreiaPena force-pushed the pix-16788-add-pix-table-on-team-table branch from a3860c1 to 5c08e93 Compare March 6, 2025 09:50
@MathGuich
Copy link
Contributor

Pour moi, le wording est ok @AndreiaPena

Petite question, du côté des invitations, est-ce que c'est normal que l'option pour les envoyer ne soit pas affichée alors que je suis administrateur ?

@AndreiaPena
Copy link
Member Author

Pour moi, le wording est ok @AndreiaPena

Petite question, du côté des invitations, est-ce que c'est normal que l'option pour les envoyer ne soit pas affichée alors que je suis administrateur ?

Tu parles de ça ?
Je vois bien les boutons

Capture d’écran 2025-03-06 à 14 07 00

@Steph0
Copy link
Contributor

Steph0 commented Mar 7, 2025

Test func panneau "equipe"

sur certifpro

  • changement de role ✅
    • 🕵🏻 j'ai note que la hauteur des colonnes n'est pas uniforme, quand le role n'a pas d'actions (genre poubelle) l'epaisseur n'est pas la meme que avec actions, ca fait un clignotement
  • variations selon si on est membre, admin,etc ✅

sur certifV3

petits trucs, mais pas bloquants

  • 🕵🏻 Pas de limites de taille / wrap sur les invitations, le tableau se decale sans limites pour la premiere colonne
    • Je ne peux plus fermer notamment les notifications (rien a voir avec cette PR)
      Screenshot_20250307_085751
  • le bandeau jaune non fermable pollue vraiment une revue graphique car il genere une barre de defilement droite

@Steph0
Copy link
Contributor

Steph0 commented Mar 7, 2025

Test func finalisation

  • Le tableau semble OK ✅ ]mais j'ai des soucis sur la modale ⚠️
    • ⚠️ la poubelle de la modale ne fonctionne pas correctement (le DELETE se fait bien, le signalement ne disparait pas

image

  • 🕵🏻 la modale a une scrollbar fantome, mais pas sur que ca soit lie a cette PR

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.

Si le truc de la poubelle sur la finalisation est ecarte, considerer que mon "comment" devient un "approve". 😄

@Libouk
Copy link
Member

Libouk commented Mar 7, 2025

Quand on navigue au clavier sur le tableau Equipe, en arrivant sur les icônes on passe d'abord sur une div (qui semble être la tooltip ou un span, j'ai un doute) qui n'est pas activable par la touche entrée. Lorsqu'on refait Tab on arrive bien sur le bouton qu'on peut activer. Pas bloquant mais surprenant
image

@AndreiaPena
Copy link
Member Author

AndreiaPena commented Mar 7, 2025

Quand on navigue au clavier sur le tableau Equipe, en arrivant sur les icônes on passe d'abord sur une div (qui semble être la tooltip ou un span, j'ai un doute) qui n'est pas activable par la touche entrée. Lorsqu'on refait Tab on arrive bien sur le bouton qu'on peut activer. Pas bloquant mais surprenant image

Ouais je te confirme c'est la tooltip, c'est vraiment pas dingue. Faudrait voir si on peut faire mieux coté Pix UI pour le coup

@AndreiaPena AndreiaPena force-pushed the pix-16788-add-pix-table-on-team-table branch from 5c08e93 to cd54a7e Compare March 7, 2025 10:55
@AndreiaPena
Copy link
Member Author

Test func finalisation

  • Le tableau semble OK ✅ ]mais j'ai des soucis sur la modale ⚠️

    • ⚠️ la poubelle de la modale ne fonctionne pas correctement (le DELETE se fait bien, le signalement ne disparait pas)

J'ai re-testé de mon coté et tout va bien :o

Enregistrement.de.l.ecran.2025-03-07.a.11.56.07.mov

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.

All good

@MathGuich MathGuich added Func Review OK PO validated functionally the PR and removed 👀 Func Review Needed Need PO validation for this functionally labels Mar 7, 2025
@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 :

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