Améliorer la gestion des permissions - Githubissues

DraftBot-A-Discord-Adventure / DraftBot

A Discord adventure bot that speaks english and french

MIT License

224 stars 61 forks source link

Améliorer la gestion des permissions #1374

Closed Pietagorh closed 12 months ago

Pietagorh commented 2 years ago

[x] Ajouter un nouveau check de permission et les messages d'erreurs associés pour la perm "voir les anciens messages"
[x] Mettre à jour les liens d'invitations avec ces nouvelles permissions dans le code du bot
[x] Mettre à jour les liens d'invitation sur le serveur du bot

Rappel de l'erreur observée (voir commentaires)

TypeError: Cannot read properties of undefined (reading 'identifier') at DraftBotShopMessage. (/home/draftbot/DraftBot-A-Discord-Adventure/dist/src/core/messages/DraftBotReactionMessage.js:122:57) at Generator.next () at fulfilled (/home/draftbot/DraftBot-A-Discord-Adventure/dist/src/core/messages/DraftBotReactionMessage.js:5:58) at runMicrotasks () at processTicksAndRejections (node:internal/process/task_queues:96:5)

BastLast commented 2 years ago

théorie actuelle: emojis externes hors du serv de base ?

Pietagorh commented 2 years ago

Ah mais vous parliez de çaaaa

Pour plus de détails : en gros sur 1 serv DraftBot arrive pas à mettre de réactions, alors qu'il a les perms pour à cause de cette erreur, que ce soit pour des emojis externes ou par défaut (parce que c'est aussi sur les reports, fights, etc)

BastLast commented 2 years ago

quel serveur ? toujours le cas actuellement ?

Feiryn commented 2 years ago

Je propose de déplacer et de hotfix si y a à nouveau le problème (avec les étapes pour reproduire)

BastLast commented 2 years ago

agree

BastLast commented 2 years ago

je redéplace en 4.0.0 parce que dans les tests ya trop de gens qui ont le problème, il vaut mieux fix ça maintenant

BastLast commented 2 years ago

oups c'est pas l'issue que je pensais

Pietagorh commented 2 years ago

quel serveur ? toujours le cas actuellement ?

318298388158480386 et oui apparemment

Pietagorh commented 1 year ago

J'ai pas eu de retour de la personne qui avait ce problème, mais chez la 2è personne qui avait eu cet exact problème c'était parce qu'il manquait la perm "voir les anciens messages" au bot, qu'on demande pas actuellement Donc il faudrait qu'on l'ajoute au lien d'invitation

BastLast commented 1 year ago

ok et faudrait qu'on affiche un message de permission manquante si le bot n'a pas la perm. Je passe l'issue dans la prochaine maj vu qu'elle est facile à faire du coup

BastLast commented 1 year ago

j'ai ajouté une tasklist à l'issue pour que ça soit plus clair pr le dev qui s'en occupe