devs-coffee / frontend-collab-chat-project

frontend for the collaborative project of a chat app
MIT License
3 stars 0 forks source link

Souci avec l'ajout de serveur #31

Closed lokkotara closed 1 year ago

lokkotara commented 1 year ago

Le 04/06/2023 : J'ai remarqué un comportement bizarre lors de ma navigation.

Je suis connecté et je suis sur la page dashboard. Si je clique sur la loupe, un champ de recherche s'ouvre, si je clique sur envoyer sans rien remplir, j'ai la liste de tous les serveurs. Si de là, je clique sur un serveur que je n'ai pas créé, je peux consulter et écrire des messages sur ce serveur sans forcément être ajouter à la liste ("join"). Si je reviens sur dashboard, ce serveur apparaît dans ma liste de serveur (temporairement jusqu'à déconnexion). Si je retourne ensuite sur ce serveur et que j'appuie maintenant sur "join", il apparaît maintenant deux fois dans ma liste de serveurs sur dashboard

WhitedogWarren commented 1 year ago

Cela fait donc 4 tâches, à voir si on crée des tickets supplémentaires.

WhitedogWarren commented 1 year ago
julienb86 commented 1 year ago

Pr en cours sur la recherche d'un serveur avec une string vide

WhitedogWarren commented 1 year ago

Petit état des lieux :

si je clique sur envoyer sans rien remplir, j'ai la liste de tous les serveurs. Done côté back. Petit fix à prévoir côté front, peut-être.

Si de là, je clique sur un serveur que je n'ai pas créé, je peux consulter et écrire des messages sur ce serveur sans forcément être ajouter à la liste ("join"). Done côté back. Petit fix à prévoir en front pour désactiver ou faire disparaître le MessageEditor.

Si je reviens sur dashboard, ce serveur apparaît dans ma liste de serveur (temporairement jusqu'à déconnexion). Done on both side. On n'affiche plus que les serveurs dont le user est membre.

Si je retourne ensuite sur ce serveur et que j'appuie maintenant sur "join", il apparaît maintenant deux fois dans ma liste de serveurs sur dashboard Done.

Résultat des courses, il reste une PR à merger côté front ( https://github.com/devs-coffee/frontend-collab-chat-project/pull/40 ) et deux fixs côté front à faire ( désactiver l'envoi d'un search avec chaîne vide et désactiver le MessageEditor si on est pas membre du serveur )

WhitedogWarren commented 1 year ago

MAJ : Il ne reste que les deux derniers fix côté front, les PR sont faites.