-
Notifications
You must be signed in to change notification settings - Fork 220
Add Elementals StoryQuest #1653
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
base: main
Are you sure you want to change the base?
Add Elementals StoryQuest #1653
Conversation
…me-template Duplicate and rename template and all files
…minijuego1 Integrar minijuego 1 y todos los cambios de Elementals
Preliminary ReviewHola! Soy Diego Alonso, Game Intern de Endless Access, haré una review preliminar de su juego.
Primera impresiónLa estética es muy atractiva; la emisión en los personajes aporta un estilo único. Input y controlesEn la primera prueba, el esquema de controles no resulta ergonómico:
Diseño de nivelesMe preocupa que el portal aparezca al inicio del juego. Errores detectados en consolaSe reportan múltiples fallas relacionadas al tileset:
Gameplay y enemigosActualmente los enemigos no tienen un diseño definido; se diferencian solo por colores de emisión. ConclusiónEl juego tiene gran potencial en estilo y diseño, pero aún requiere pulir varios aspectos técnicos y de jugabilidad. Información en el Pull RequestPor último, es muy importante que agreguen más información en la descripción del PR.
Esto se ilustra mejor en el ejemplo de PR en la guía oficial: |
Gatorrante
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Les recuerdo que No deben modificar los archivos afuera de la carpeta de su Storyquest pues puede causar conflictos con el resto del juego, así mismo, a que se debe el cambio de la clase? pasó de ser "Projectile" a "Projectiles" por algún motivo? recomiendo restaurar el archivo.
Gatorrante
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Restauren el archivo o corrijan el formato.
Gatorrante
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
También restauren storybook.tscn
Gatorrante
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Es recomendable, y una buena práctica, cambiar el nombre del archivo utilizando un nombre claro y descriptivo.
Evita usar espacios en el nombre y no lo dejes como "WhatsApp Audio".
Esto ayuda a mantener la consistencia, prevenir posibles problemas en el proyecto y facilitar la gestión de los assets.
Gatorrante
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Es recomendable, y una buena práctica, cambiar el nombre del archivo utilizando un nombre claro y descriptivo. Evita usar espacios en el nombre y no lo dejes como
"WhatsApp Audio". Esto ayuda a mantener la consistencia, prevenir posibles problemas en el proyecto y facilitar la gestión de los assets.
Igualmente aquí.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
@KevinSplits thanks for the submission! As @Gatorrante says, the very first thing to do here for applying to the submission guidelines is to provide a good title and description. |
|
@KevinSplits and team, we decided to keep all StoryQuests PRs open for 3 weeks. Upon that time, if we don't hear back from you, we'll be closing them. If you can, please apply our suggestions so we can merge the PR. Thanks! |



No description provided.