Skip to content

Conversation

@TheWeebCrafter10010
Copy link

Juego el valiente soldadito con el puzzle de hielo

@TheWeebCrafter10010 TheWeebCrafter10010 requested review from a team as code owners November 18, 2025 23:41
Copy link
Collaborator

@manuq manuq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi,
I didn't have the chance to play the game yet. But I would like to tell you that the bear graphic is cute. Was it done with an LLM?

Please check with your mentor the very first condition we have for the Story Quest submission:

the PR must be strictly confined to creating and modifying files only within the directory structure that belongs to the StoryQuest. PRs that extend beyond this will be rejected.

@manuq manuq added the StoryQuest New StoryQuests or updates to existing ones label Nov 19, 2025
@TheWeebCrafter10010
Copy link
Author

Hi, I didn't have the chance to play the game yet. But I would like to tell you that the bear graphic is cute. Was it done with an LLM?

Please check with your mentor the very first condition we have for the Story Quest submission:

the PR must be strictly confined to creating and modifying files only within the directory structure that belongs to the StoryQuest. PRs that extend beyond this will be rejected.

Hi! The bear was drawn by one of my partners, so no LLM used! I think I modified some files outside our directory because i wanted to check and prove some mechanics so I'll update this branch as soon as posible!

@manuq
Copy link
Collaborator

manuq commented Nov 19, 2025

Hi, I didn't have the chance to play the game yet. But I would like to tell you that the bear graphic is cute. Was it done with an LLM?
Please check with your mentor the very first condition we have for the Story Quest submission:

the PR must be strictly confined to creating and modifying files only within the directory structure that belongs to the StoryQuest. PRs that extend beyond this will be rejected.

Hi! The bear was drawn by one of my partners, so no LLM used!

Interesting! Which software are they using?

@github-actions
Copy link

Play this branch at https://play.threadbare.game/branches/Gamelab5-UTP-Grupo8-2/el_valiente_soldadito.

(This launches the game from the start, not directly at the change(s) in this pull request.)

@Omar071201 Omar071201 self-requested a review November 26, 2025 15:11
@manuq
Copy link
Collaborator

manuq commented Dec 2, 2025

Quest review:

The "slip on ice" mechanic is fantastic! (Here is a direct link to play it) When touching the ice, the player moves in that direction until a wall is hit. Great idea to combine it with the guards from the stealth game. I like the ice tileset too!

image

This is the single minigame in the SQ, plus intro and otro. I think that's great. If you could make a bit of extra effort, we could merge this StoryQuest. It will be a great addition to the main game!

Technical review:

As mentioned in #1637 (review) the StoryQuest must be adding a single folder, and all other files in the game must be restored.

The quest appears as "MyQuest" in the StoryBook. Please adjust the title.

There are many warnings about duplicated UIDs when opening the project:

image

This is usually because of files duplicated outside Godot. If you duplicate them within Godot, the editor makes sure to update their UIDs.

@TheWeebCrafter10010 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!

@manuq manuq changed the title El valiente soldadito Add El valiente soldadito StoryQuest Dec 2, 2025
Copy link
Collaborator

@manuq manuq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See the requests above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

StoryQuest New StoryQuests or updates to existing ones

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants