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

Suppression du message pour rejoindre un groupe une fois fermé / dissout #151

Open
rick-gnous opened this issue Apr 4, 2024 · 7 comments
Labels
enhancement New feature or request good first issue Good for newcomers groupe priorité basse

Comments

@rick-gnous
Copy link
Member

Une fois un groupe dissout ou terminer, il faudrait ou supprimer les boutons / le post, ou les griser.

@rick-gnous rick-gnous added enhancement New feature or request good first issue Good for newcomers priorité basse groupe labels Apr 4, 2024
@0xN0x
Copy link
Contributor

0xN0x commented Apr 4, 2024

Et rajouter un check dans joinGroup et leaveGroup pour vérifier si le groupe avec cet id existe bien avant de faire quoi que ce soit. Peu importe le choix fait dans #152

@Kekwel
Copy link
Collaborator

Kekwel commented Apr 4, 2024

Si le groupe est dissout, le message avec ses boutons sont bien supprimés

@rick-gnous
Copy link
Member Author

Si le groupe est dissout, le message avec ses boutons sont bien supprimés

il faudrait qu'on voit si on supprime le message à la fin d'un groupe ou si on le grise

@0xN0x
Copy link
Contributor

0xN0x commented Apr 4, 2024

Si le groupe est dissout, le message avec ses boutons sont bien supprimés

il faudrait qu'on voit si on supprime le message à la fin d'un groupe ou si on le grise

Modifier le message va juste garder de la masse inutile dans un salon déjà assez chargé sans ça, je pense que le supprimé est plus cohérent, dans tous les cas il ne servira a rien dans le futur et ça peut permettre de faire redescendre certains groupes plus anciens qui sont encore ouverts.

Est-ce que les utilisateurs ont besoin d'un archivage de ces groupes de leur coté pour une raison particulière ?

@rick-gnous
Copy link
Member Author

+1, et non, @Kekwel voulait faire des stats dessus dans mes souvenirs, les utilisateurs finaux ne le voient pas

@0xN0x
Copy link
Contributor

0xN0x commented Apr 4, 2024

Dans ce cas là juste faire les stats sur la base est suffisant, pas besoin de les garder dans un salon discord je pense

@Kekwel
Copy link
Collaborator

Kekwel commented Apr 4, 2024

+1, et non, @Kekwel voulait faire des stats dessus dans mes souvenirs, les utilisateurs finaux ne le voient pas

yep, on a déjà qques nombres dans la collection users
image

@rick-gnous rick-gnous changed the title Griser les boutons d'un groupe fermé Suppression du message pour rejoindre un groupe une fois fermé / dissout Apr 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers groupe priorité basse
Projects
None yet
Development

No branches or pull requests

3 participants