Skip to content
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

Document wonder_talk2 and remove the ones in Thieves' hideout #2232

Merged
merged 2 commits into from
Sep 30, 2024

Conversation

GSKirox
Copy link
Collaborator

@GSKirox GSKirox commented May 30, 2024

Wonder_talk2 is an actor that displays a textbox when near a certain spot, either automatically or by pressing A (button turns to Check). They're mostly present at the start of Shadow Temple and in every room of Gerudo Training Ground, and have been removed in randomizer years ago for convenience.

This PR adds information on how they are removed, move this process out of Patches.py in Cutscenes.py (it's not really cutscenes but it fits here imo), and use the same method to remove the ones in Thieves' Hideout when you get near a Carpenter's cell :
Hey, you! Young man, over there!
Look over here, inside the cell!

The wonder_talk2 at the Crater Fairy entrance was patched differently from the other ones, but as far as i can see, there's no real advantage to this, so i changed it to use the same method as the other ones.

@r0bd0g
Copy link

r0bd0g commented May 30, 2024

I'm going to continue to jumpslash into these for years to come.

@GSKirox
Copy link
Collaborator Author

GSKirox commented May 30, 2024

You have to jumpslash anyway to power crouch stab during the fight so it doesn't lose time overall 😄

@fenhl fenhl added Type: Enhancement New feature or request Status: Needs Review Someone should be looking at it Status: Needs Testing Probably should be tested Status: Under Consideration Developers are considering whether to accept or decline the feature described Component: Patching Affects the patching of the ROM Type: Maintenance Code style, infrastructure, updating dependencies labels May 30, 2024
Copy link
Collaborator

@fenhl fenhl left a comment

Choose a reason for hiding this comment

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

Just a code style nit, looks good other than that.

Cutscenes.py Outdated Show resolved Hide resolved
@fenhl fenhl added the Status: Waiting for Author Changes or response requested label Jun 5, 2024
Co-authored-by: Fenhl <fenhl@fenhl.net>
@fenhl fenhl removed Status: Waiting for Author Changes or response requested Status: Needs Review Someone should be looking at it labels Jun 6, 2024
@fenhl fenhl added Status: Waiting for Release This PR is ready for merge, but we're holding off on it until after the next release and removed Status: Under Consideration Developers are considering whether to accept or decline the feature described Status: Needs Testing Probably should be tested labels Jul 15, 2024
@fenhl
Copy link
Collaborator

fenhl commented Sep 30, 2024

Merged in 8de7593.

@fenhl fenhl closed this Sep 30, 2024
@fenhl fenhl removed the Status: Waiting for Release This PR is ready for merge, but we're holding off on it until after the next release label Sep 30, 2024
@fenhl
Copy link
Collaborator

fenhl commented Sep 30, 2024

Got the PRs mixed up here somehow.

@fenhl fenhl reopened this Sep 30, 2024
@fenhl fenhl added this to the next milestone Sep 30, 2024
@fenhl fenhl merged commit 0568bb3 into OoTRandomizer:Dev Sep 30, 2024
6 checks passed
@GSKirox GSKirox deleted the wondertalk2 branch October 1, 2024 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Patching Affects the patching of the ROM Type: Enhancement New feature or request Type: Maintenance Code style, infrastructure, updating dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants