Undertimes - Githubissues

HollishKid / Gaia

Espace de collaboration du serveur Gaïa MMORPG.

6 stars 2 forks source link

Undertimes #117

Closed Undertimes closed 7 years ago

Undertimes commented 7 years ago

Le pnj et ses quêtes journalières sont prêtes! Les différentes quêtes fonctionnent (tests réussis sur le serveur dev) et le pnj se débloque pour les joueurs ayant atteint le grade d'écuyer.

HollishKid commented 7 years ago

Ce PR m'a l'air très bien, mais je me pose quelques questions.

n'y a-t-il pas un niveau de dossiers en trop ? UnderTimes > Quêtes sanctuaire Il me semble que c'est théoriquement possible, je me pose plutôt des questions de "facilité" ;
ne va-t-il pas y avoir de souci dans la définition des tags, objectifs etc. avec un espace et un caractère spécial dans un nom de package ? On aurait du UnderTimes.Quêtes Sancutaire.nom_du_tag ; ce qui me parait pas optimal ?
a plus long terme on n'aurait-t-on pas du mal à faire référence à ce package ?
une fois en place on ne peut pas le modifier, puisque les joueur auront déjà des tags etc. du format que j'ai donné plus haut.

crashoz commented 7 years ago

Effectivement je n'avais pas vu, ce serait bien mieux d'avoir les quêtes dans un seul dossier UnderTimes je suis d'accord.

Undertimes commented 7 years ago

En effet, il serai plus efficace de ne pas avoir de sous dossier, sous tout les angles.

Un autre détail que j'ai oublié de préciser est que le pnj 'joseph' situé dans le château royal peut être merge avec un autre pnj lié au sanctuaire, par exemple l'un des pnj du second étage, étant donné que ces quêtes ne sont accessibles qu'aux écuyers. Si vous avez une idée, n'hésitez pas.

Sent from my Sony D5803 using FastHub

Undertimes commented 7 years ago

Le sous-dossier est supprimé, y a-t-il autre chose à modifier?

Undertimes commented 7 years ago

Modifications appliquées, ça devrait être bon pour ce que Hollish a relevé.

Undertimes commented 7 years ago

Plus que l'approbation de @HollishKid et je merge.