[FEATURE] Afficher les invitations de membre en attente d'une organisation sur Pix Admin (PIX-397). #3651

AndreiaPena · 2021-10-25T16:23:12Z

🎃 Problème

Actuellement sur Pix Admin, il n'y a aucun moyen de voir les invitations en attente des organisations.

🦇 Solution

Dans la page de détail d'une organisation, ajouter un onglet "Invitations" et y lister toutes les invitations en attente, avec l'adresse e-mail, le rôle et la date (et heure) de dernier envoi de mail.

🕸️ Remarques

On tri par date de dernier envoi (updatedAt), la plus récente d'abord. (on le fait coté API, sur la méthode du repository, également appelé sur Pix Orga. On le fait également coté front sur Pix Admin, pour mettre à jour le tableau dès qu'on ajoute une nouvelle invitation)
Déplacement de l’action “Inviter un membre' dans ce nouvel onglet.
Renommage de l'onglet "Membre" en "Equipe"
Commit pour séparer les titres d'onglet par un séparateur | .
Renommer le label "Sans rôle" par "Automatique" => suite à la démo

👻 Pour tester

Se connecter sur pix Admin ( pixmaster@example.net)
Aller sur une organisation au hasard
Aller sur l'onglet Invitations

S'assurer que le tri est effectif, le plus récent en haut.
Si pas de rôle, que ça affiche "-"
Si pas d'invitations, on voit le message suivant : "Aucune invitation en attente"
Constater que le champ pour inviter un membre se trouve désormais dans cet onglet, et plus dans "Equipe" (Anciennement "Membres")

pix-service · 2021-10-25T16:28:39Z

I'm deploying this PR to these urls:

App (.fr): https://app-pr3651.review.pix.fr
App (.org): https://app-pr3651.review.pix.org
Orga: https://orga-pr3651.review.pix.fr
Certif: https://certif-pr3651.review.pix.fr
Admin: https://admin-pr3651.review.pix.fr
API: https://api-pr3651.review.pix.fr/api/

Please check it out!

admin/tests/integration/components/organization-invitations_test.js

annemarie35 · 2021-11-03T10:29:28Z

admin/app/components/organization-team-actions-section.hbs

@@ -1,4 +1,4 @@
-<section class="page-section mb_10 organization-members-section">
+<section class="page-section mb_10 organization-team-section">


yeah ! le souci du détail ^^

admin/tests/unit/components/organization-invitations_test.js

annemarie35 · 2021-11-03T11:05:50Z

❌ J'ai pu observer le comportement suivant :

je suis connectée en tant que pix master
je vais voir la liste des invitation de l'organisation 1 https://admin-pr3651.review.pix.fr/organizations/1/invitations
je vois bien une invitation sans rôle envoyée à pro.invited@example.net le 01/01/2020 - 01:00
je crée une nouvelle organisation https://admin-pr3651.review.pix.fr/organizations/10000001/team
je vais voir la liste des invitations
je vois l'invitation de l'organisation que j'ai consulté précédemment une invitation sans rôle envoyée à pro.invited@example.net le 01/01/2020 - 01:00

je vide le cache de mon navigateur
je vois bien qu'il n'y a aucune invitation en attente pour l'organisation 10000001 que j'ai créé Aucune invitation en attente
si j'ajoute un membre à cette nouvelle orga, puis que je retourne sur la première organisation https://admin-pr3651.review.pix.fr/organizations/1/invitations
je vois deux invitations

AndreiaPena · 2021-11-03T13:08:46Z

❌ J'ai pu observer le comportement suivant :

Ok au départ j'ai cru que la création d'une nouvelle orga ajoutait automatique une organization invitation mais c'est pas le cas.
En fait c'est ember qui garde en "mémoire" les infos de la précédente organisation que nous sommes allés visiter.
Il embarque les précédentes données au lieu de reset pour n'afficher que les données reçues par l'API lorsque l'on va dans l'onglet d'une autre orga.
Et comme @annemarie35 le signale, dès qu'on rafraîchit le navigateur, l'affichage est correct.

Je vais voir ce qui cause ça, peut être qu'un reload est nécessaire ? Ou que le findAll n'est pas adapté ?

Anne-Gaelle-S

Pas testé fonctionnellement mais pour moi le code est good !
A part 1 commit un peu fouillis j'ai trouvé super le découpage des commits. J'ai l'impression qu'il y a eu une vision nette de la PR et une amélioration sur le découpage commit par commit.
Et donc je voulais le souligner, bravo ça se lit comme du petit lait ! 🥛 😍

Anne-Gaelle-S · 2021-11-03T16:21:11Z

api/tests/integration/application/organizations/index_test.js

  describe('POST /api/organizations', function () {
    it('should exist', async function () {
      // given
      const method = 'POST';
      const url = '/api/organizations';

+      sinon.stub(securityPreHandlers, 'checkUserHasRolePixMaster').callsFake((request, h) => h.response(true));


admin/app/components/organization-invitations.hbs

Anne-Gaelle-S · 2021-11-03T16:31:01Z

admin/tests/unit/components/organization-invitations_test.js

@@ -0,0 +1,31 @@
+import { module, test } from 'qunit';


remarque
J'ai l'impression que les modifs ici n'auraient pas due atterrir dans le commit "[ADMIN] Move a member's invitation form to the invitations tab" ?
En tout cas ce commit me paraît un peu fouilli comparés aux autres, c'est dommage c'était si beau à lire jusqu'à là 🤩

Anne-Gaelle-S · 2021-11-03T16:32:08Z

admin/app/controllers/authenticated/organizations/get/invitations.js

@@ -21,14 +21,15 @@ export default class InvitationsController extends Controller {
      return;
    }

+    const organizationInvitation = this.store.createRecord('organization-invitation', { email, lang, role });


🎉
Je trouve ça tellement stylé les commits avec un nom à rallonge et 3 lignes de codes... je sais pas pourquoi 😎

Anne-Gaelle-S · 2021-11-03T16:33:52Z

admin/app/models/organization-invitation.js

+      case 'MEMBER':
+        return 'Membre';
+      default:
+        return '-';


question
Quand il a pas de rôle c'est qu'il n'a pas encore cliqué sur l'invitation ?
Si oui, est-ce que ça serait plus parlant de mettre "En attente d'acceptation de l'invitation" ? (je sais pas du tout je propose)

…ot pass

AndreiaPena added Development in progress team-acces labels Oct 25, 2021

AndreiaPena self-assigned this Oct 25, 2021

AndreiaPena force-pushed the pix-397-display-pending-invitations-in-pix-admin branch from 7abd16d to fd847f6 Compare October 25, 2021 16:25

AndreiaPena force-pushed the pix-397-display-pending-invitations-in-pix-admin branch 7 times, most recently from 8868256 to 6d447b7 Compare October 28, 2021 15:47

AndreiaPena changed the title ~~[FEATURE] Afficher les invitations de membre d'une organisation sur Pix Admin (PIX-397).~~ [FEATURE] Afficher les invitations de membre en attente d'une organisation sur Pix Admin (PIX-397). Oct 28, 2021

AndreiaPena force-pushed the pix-397-display-pending-invitations-in-pix-admin branch from 6d447b7 to 9f41c48 Compare October 29, 2021 08:38

yschoueri force-pushed the pix-397-display-pending-invitations-in-pix-admin branch 2 times, most recently from 1c8cbca to 8532eaf Compare November 2, 2021 09:17

AndreiaPena force-pushed the pix-397-display-pending-invitations-in-pix-admin branch 13 times, most recently from 7d8ed0c to 933eb20 Compare November 2, 2021 15:45

AndreiaPena removed the Development in progress label Nov 2, 2021

annemarie35 reviewed Nov 3, 2021

View reviewed changes

admin/tests/integration/components/organization-invitations_test.js Outdated Show resolved Hide resolved

annemarie35 reviewed Nov 3, 2021

View reviewed changes

admin/tests/unit/components/organization-invitations_test.js Show resolved Hide resolved

AndreiaPena force-pushed the pix-397-display-pending-invitations-in-pix-admin branch from 3e1b91b to 618475f Compare November 3, 2021 13:23

Anne-Gaelle-S approved these changes Nov 3, 2021

View reviewed changes

AndreiaPena force-pushed the pix-397-display-pending-invitations-in-pix-admin branch from 6672890 to 908a445 Compare November 4, 2021 10:24

AndreiaPena added 13 commits November 4, 2021 11:25

[API] Add admin route to get pending invitations

5d1e957

[API][SR] Delete beforeEach in index test

44cdc64

[API] Add role attribute in organization invitation serializer

b2c7aaa

[API] Return organization-invitations from most recent to oldest

e61cedc

[ADMIN] Add invitation route

5a7ceea

[ADMIN] Display organization invitations

0206bbb

[ADMIN] Rename Members tab to Team

01a0277

[ADMIN] Move a member's invitation form to the invitations tab

e5acd7b

[ADMIN] Do not post a new entry on the board if the invitation does n…

6c439b3

…ot pass

[SR] Fix mocha rules lint

96e18da

[ADMIN][SR] Add separator on page title

28950fc

[ADMIN] Translate invitation roles

932a935

[ADMIN] Change label

97d7720

AndreiaPena force-pushed the pix-397-display-pending-invitations-in-pix-admin branch from 908a445 to 41e7ad4 Compare November 4, 2021 10:25

[ADMIN] Review

1cb7ceb

AndreiaPena force-pushed the pix-397-display-pending-invitations-in-pix-admin branch from 41e7ad4 to 1cb7ceb Compare November 4, 2021 10:43

AndreiaPena added 🚀 Ready to Merge and removed 👀 Tech Review Needed labels Nov 4, 2021

pix-service-auto-merge merged commit cc9a1a1 into dev Nov 4, 2021

pix-service-auto-merge deleted the pix-397-display-pending-invitations-in-pix-admin branch November 4, 2021 10:53

yschoueri mentioned this pull request Nov 8, 2021

[BUGFIX] Eviter les doublons quand on re-invite un membre sur Pix Admin (PIX-3802). #3701

Merged

octo-topi mentioned this pull request Jan 3, 2022

[TECH Ajouter la règle de lint mocha/no-exclusive-tests (PIX-4085). #3887

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[FEATURE] Afficher les invitations de membre en attente d'une organisation sur Pix Admin (PIX-397). #3651

[FEATURE] Afficher les invitations de membre en attente d'une organisation sur Pix Admin (PIX-397). #3651

AndreiaPena commented Oct 25, 2021 •

edited

Loading

pix-service commented Oct 25, 2021

annemarie35 Nov 3, 2021

annemarie35 commented Nov 3, 2021 •

edited

Loading

AndreiaPena commented Nov 3, 2021 •

edited

Loading

Anne-Gaelle-S left a comment

Anne-Gaelle-S Nov 3, 2021

Anne-Gaelle-S Nov 3, 2021

Anne-Gaelle-S Nov 3, 2021

Anne-Gaelle-S Nov 3, 2021

		@@ -1,4 +1,4 @@
		<section class="page-section mb_10 organization-members-section">
		<section class="page-section mb_10 organization-team-section">

[FEATURE] Afficher les invitations de membre en attente d'une organisation sur Pix Admin (PIX-397). #3651

[FEATURE] Afficher les invitations de membre en attente d'une organisation sur Pix Admin (PIX-397). #3651

Conversation

AndreiaPena commented Oct 25, 2021 • edited Loading

🎃 Problème

🦇 Solution

🕸️ Remarques

👻 Pour tester

pix-service commented Oct 25, 2021

annemarie35 Nov 3, 2021

Choose a reason for hiding this comment

annemarie35 commented Nov 3, 2021 • edited Loading

AndreiaPena commented Nov 3, 2021 • edited Loading

Anne-Gaelle-S left a comment

Choose a reason for hiding this comment

Anne-Gaelle-S Nov 3, 2021

Choose a reason for hiding this comment

Anne-Gaelle-S Nov 3, 2021

Choose a reason for hiding this comment

Anne-Gaelle-S Nov 3, 2021

Choose a reason for hiding this comment

Anne-Gaelle-S Nov 3, 2021

Choose a reason for hiding this comment

AndreiaPena commented Oct 25, 2021 •

edited

Loading

annemarie35 commented Nov 3, 2021 •

edited

Loading

AndreiaPena commented Nov 3, 2021 •

edited

Loading