-
Notifications
You must be signed in to change notification settings - Fork 220
Actualizacion del minijuego 2 #1583
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?
Actualizacion del minijuego 2 #1583
Conversation
…-and-rename-template-grupo2 Duplicacion y creacion de la carpeta Grupal
…folder-grupo-2-ecos-de-media-noche Template folder grupo 2 ecos de media noche
…folder-grupo-2-ecos-de-media-noche Template folder grupo 2 ecos de media noche
…folder-grupo-2-estation-echo-9 Update the folder names to match the updates in the GDD
Minigame 2 Advance
…re-quests-location Revert: Move lore_quests back to correct location
…preview-2 minigame preview 2
Actualizacion de la Intro Version 1
Creación del nivel 1
…ara hacer el juego mas sencillo
Mejora en la luz dnimaica en el minijuego y posiciones estrategicas p…
|
Play this branch at https://play.threadbare.game/branches/Game-Lab-5-0-UTP-Group-6-Team-2/actualizacion-del-minijuego-2. (This launches the game from the start, not directly at the change(s) in this pull request.) |
manuq
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.
This is looking great. Please improve the submission according to this guide
In particular:
- The Title field should briefly mention what is included in the StoryQuest, and is ideally written in English.
- The Description should include a brief outline of the work included in the PR. As the story quest content is added to the game via the PRs, this should include what concepts/stories it is built around and what are the main gameplay elements. It is OK to submit this in Spanish. Attribution sections titled as Assets, Engineering, Game Design, and Production & Management. These should reference GitHub usernames of the contributors.
Check how this other team is doing it: #1561
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.
Please add the following two lines at the top of the shader file. Otherwise the license check fails.
| // SPDX-FileCopyrightText: The Threadbare Authors | |
| // SPDX-License-Identifier: MPL-2.0 | |
| shader_type canvas_item; |
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.
Please revert the change of this file, by restoring scenes/tileset.tres from upstream main. Thanks!
|
Quest review: I was able to beat the level! I think that you have a great single-level StoryQuest here. I love the level design. And how the lights dim, making it a bit more challenging on one hand, and a bit easier to spot the guards on the other hand. The use of a shader is a very simple and effective aestheric decision. If you can, please apply my suggestions above because I would love to this this quest in the main game. Don't worry about creating 2 more minigames, you can provide the 2 other threads with dialogue by following this tutorial.
|


se actualizo el minijuego 2