Open BastLast opened 4 months ago
normalement oui, mais y'a un truc qui me plait pas et que j'ai pas trouvé de moyen de fix :
Si le core répond trop tardivement à une requête de discord non deferred, ca va réagir pareil que lorsque le bot n'a pas les perms suffisantes (ou quand y'a un conflit sur les commandes d'envoi). La seconde raison n'est normalement pas censé survenir en prod, mais y'a quand même une mauvaise info user de pourquoi ca a planté
normalement oui, mais y'a un truc qui me plait pas et que j'ai pas trouvé de moyen de fix :
Si le core répond trop tardivement à une requête de discord non deferred, ca va réagir pareil que lorsque le bot n'a pas les perms suffisantes (ou quand y'a un conflit sur les commandes d'envoi). La seconde raison n'est normalement pas censé survenir en prod, mais y'a quand même une mauvaise info user de pourquoi ca a planté
Si c'est pas super mega simple a fix, go fermer cette issue et en ouvrir une nouvelle pr fix ce que tu as dit après la sortie de la v5
normalement oui, mais y'a un truc qui me plait pas et que j'ai pas trouvé de moyen de fix :
Si le core répond trop tardivement à une requête de discord non deferred, ca va réagir pareil que lorsque le bot n'a pas les perms suffisantes (ou quand y'a un conflit sur les commandes d'envoi). La seconde raison n'est normalement pas censé survenir en prod, mais y'a quand même une mauvaise info user de pourquoi ca a planté
On ne peut pas faire en sorte d'envoyer un paquet "MissingPermissionPacket" pour afficher l'erreur de perm, et si pas de réponse de Core on affiche un autre type d'erreur ?
normalement oui, mais y'a un truc qui me plait pas et que j'ai pas trouvé de moyen de fix : Si le core répond trop tardivement à une requête de discord non deferred, ca va réagir pareil que lorsque le bot n'a pas les perms suffisantes (ou quand y'a un conflit sur les commandes d'envoi). La seconde raison n'est normalement pas censé survenir en prod, mais y'a quand même une mauvaise info user de pourquoi ca a planté
On ne peut pas faire en sorte d'envoyer un paquet "MissingPermissionPacket" pour afficher l'erreur de perm, et si pas de réponse de Core on affiche un autre type d'erreur ?
le problème c'est que c'est forcément une erreur de discord dans ce cas, core fonctionne très bien dans les 3 cas ci dessus, juste que l'un est un problème user (pas les bonnes perms dans le salon), le second erreur réseau (core a bien recu l'info, mais l'info retour a mis trop de temps), la 3e une erreur dev
normalement oui, mais y'a un truc qui me plait pas et que j'ai pas trouvé de moyen de fix : Si le core répond trop tardivement à une requête de discord non deferred, ca va réagir pareil que lorsque le bot n'a pas les perms suffisantes (ou quand y'a un conflit sur les commandes d'envoi). La seconde raison n'est normalement pas censé survenir en prod, mais y'a quand même une mauvaise info user de pourquoi ca a planté
On ne peut pas faire en sorte d'envoyer un paquet "MissingPermissionPacket" pour afficher l'erreur de perm, et si pas de réponse de Core on affiche un autre type d'erreur ?
le problème c'est que c'est forcément une erreur de discord dans ce cas, core fonctionne très bien dans les 3 cas ci dessus, juste que l'un est un problème user (pas les bonnes perms dans le salon), le second erreur réseau (core a bien recu l'info, mais l'info retour a mis trop de temps), la 3e une erreur dev
Ah oui c'est discord qui génère l'erreur. Mais je crois que je n'ai pas compris l'objet de l'issue alors. C'est quoi le bug a fix (ou qui a été fixé) dans cette issue ? Car les 3 que tu viens de me dire me semblent déjà gérés
le bug géré ici était le fait que si une erreur survenait, alors le langage de l'interaction était inconnu, donc ca plantait bien comme il faut lors d'un envoi de message raté. Cependant lorsqu'il y a une erreur, ca émet une erreur de manque de perms dans les 3 cas, ce qui est pas top pour l'utilisateur qui tombe dessus (alors que ca peut juste être un problème de latence)
Pourquoi on envoie un problème de perm dans le cas numéro 2 et 3 ? Dans le cas numéro 2, si on n'a pas de réponse de Core on va juste avoir l'interaction qui échoue côté discord. Dans le cas numéro 3 (qui ne devrait pas arriver), si y a un crash console normalement y un paquet erreur qui est retourné, et si ça ne répond pas on retourne dans le cas 2
Pourquoi on envoie un problème de perm dans le cas numéro 2 et 3 ? Dans le cas numéro 2, si on n'a pas de réponse de Core on va juste avoir l'interaction qui échoue côté discord. Dans le cas numéro 3 (qui ne devrait pas arriver), si y a un crash console normalement y un paquet erreur qui est retourné, et si ça ne répond pas on retourne dans le cas 2
les 3 tombent au même endroit en cas d'erreur, et j'ai pas eu le temps ni la patience de chercher à split les cas (cf https://github.com/DraftBot-A-Discord-Adventure/DraftBot/blob/71ee4aa270eda1265c9cb7766872d46d15e33e5f/Discord/src/messages/DraftbotInteraction.ts#L227)
normalement oui, mais y'a un truc qui me plait pas et que j'ai pas trouvé de moyen de fix : Si le core répond trop tardivement à une requête de discord non deferred, ca va réagir pareil que lorsque le bot n'a pas les perms suffisantes (ou quand y'a un conflit sur les commandes d'envoi). La seconde raison n'est normalement pas censé survenir en prod, mais y'a quand même une mauvaise info user de pourquoi ca a planté
Si c'est pas super mega simple a fix, go fermer cette issue et en ouvrir une nouvelle pr fix ce que tu as dit après la sortie de la v5
Vu ce qui a été écrit, je pense qu'il faut quand même fix ça. Pas ouf du tout que l'utilisateur reçoive ce genre d'erreur car il va passer du temps à checker les perms pour rien. Et nous on perdra du temps en support car l'erreur n'aura aucun lien avec un problème de perm
normalement oui, mais y'a un truc qui me plait pas et que j'ai pas trouvé de moyen de fix : Si le core répond trop tardivement à une requête de discord non deferred, ca va réagir pareil que lorsque le bot n'a pas les perms suffisantes (ou quand y'a un conflit sur les commandes d'envoi). La seconde raison n'est normalement pas censé survenir en prod, mais y'a quand même une mauvaise info user de pourquoi ca a planté
Si c'est pas super mega simple a fix, go fermer cette issue et en ouvrir une nouvelle pr fix ce que tu as dit après la sortie de la v5
Vu ce qui a été écrit, je pense qu'il faut quand même fix ça. Pas ouf du tout que l'utilisateur reçoive ce genre d'erreur car il va passer du temps à checker les perms pour rien. Et nous on perdra du temps en support car l'erreur n'aura aucun lien avec un problème de perm
Fair enough
@romain22222 on peut close ça non ? tu l'as fix non ?