DraftBot-A-Discord-Adventure / DraftBot

A Discord adventure bot that speaks english and french
MIT License
229 stars 61 forks source link

Retour du mont Celestrum pour le croisement des Destins #1585

Open QQtin3 opened 2 years ago

QQtin3 commented 2 years ago

Description du bug.

On peut faire Croisement des destins --> Mont Celestrum --> Croisement des destins

Selon vous, quelle est l'importance de ce bug ?

🙂 Faible

Reproduction du bug.

Passer par le Croisement des Destins aller au Mont Celestrum et revenir

Captures d'écran

IMG_20221108_073925 IMG_20221108_073938

Sur quel shard avez vous rencontré le bug ?

Shard 3

BastLast commented 2 years ago

Normal c'est forcé par le texte de l'event

BastLast commented 2 years ago

Si il partait pas par la le texte serait pas logique

romain22222 commented 2 years ago

en relisant l'event attentivement, je peux confirmer que c'est complètement débile que ca redirige vers le croisement des destins

on "traverse" la forêt sans difficulté donc deux problèmes :

de plus, rien n'indique qu'on aille au croisement ensuite, donc il n'est pas DU TOUT forcé, tu pourrais aller soit au chemin des loups, soit à l'étendue. Ce qui prouve bien que c'est soit un bug niveau event (donc à modifier) soit un bug niveau logique de direction (le demi-tour étant ici bien un bug vu que ca ne devrait pas logiquement revenir en arrière, qui plus est que j'ai dit vouloir continuer la grimpe)

BastLast commented 2 years ago

Je suis contre toute modification du code derrière, mais il y a en effet un problème de logique ici, go edit les paramètres de l'event pour corriger cela.

romain22222 commented 2 years ago

Si on donne plusieurs possibilités de sortie, et que l'event dépend de la zone précédente (comme ici), va falloir rajouter un check pour le choix d'event du coup

romain22222 commented 2 years ago

on peut pas juste éditer les params de l'event pour fix le problème

BastLast commented 2 years ago

Bah si, en changeant le texte de l'event ou en le déplaçant

BastLast commented 2 years ago

Maybe créer un nouveau lieu ?

romain22222 commented 2 years ago

draft qui veut VRAIMENT pas toucher au code ca me fume, l'event est juste correct niveau texte, c'est la logique derrière qui passe pas la

QQtin3 commented 2 years ago

Juste à remove le faire de pouvoir back non ?

romain22222 commented 2 years ago

Juste à remove le faire de pouvoir back non ?

y'a ca et le fait de pouvoir tomber sur cet event QUE si tu viens bien d'une forêt

BastLast commented 2 years ago

Juste à remove le faire de pouvoir back non ?

le fait de pouvoir back n'est pas forcément un problème pr d'autres évents tho

BastLast commented 2 years ago

Juste à remove le faire de pouvoir back non ?

y'a ca et le fait de pouvoir tomber sur cet event QUE si tu viens bien d'une forêt

je suis d'accord que ceci necessite en effet une modif du code.

BastLast commented 2 years ago

par contre je suis chaud qu'on garde la roadmap actuelle :

Prochaine release : rework reward de combat Release suivante : PVE

et après ça bosser sur le rework de la commande report avec au passage la prise en compte des stats ou des objets pour les issues comme ça on fait 2 pierres d'un coup

pwaillette commented 1 year ago

Toujours d'actualité, je présume?

QQtin3 commented 1 year ago

Effectivement c'est probable, road to après la V5 si on veut même s'en occuper un jour, m'enfin c'est assez peu problématique pour y consacrer un temps considérable