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

Fix for "Fewer Tunic Requirements" allowing child access to Fire Temple Boulder Maze #3273

Merged

Conversation

Malkierian
Copy link
Contributor

@Malkierian Malkierian commented Oct 9, 2023

Previously to "Fewer Tunic Requirements" trick being added, FireTimer would require being adult via CanUse(GORON_TUNIC). Adding the trick opened up a logic bug for items to be placed in lower maze with expectation for child collection. However, there's no glitchless way to traverse the Shortcut Room before it as child. This adds IsAdult to the check for traversing the climb up to boulder maze to restrict access for items expected as child while maintaining first floor access during dungeon entrance shuffle.

Build Artifacts

…ndo logic to preven placing items in lower maze expecting child to get them.
…aze so child access with dungeon shuffle remains intact.

Also changed the tooltip for Fewer Tunic Requirements to reflect the current status of what the trick enables.
@Malkierian Malkierian changed the title Add IsAdult to Fewer Tunic Requirements check for FireTimer rando logic Fix for "Fewer Tunic Requirements" allowing child access to Fire Temple Boulder Maze Oct 15, 2023
@garrettjoecox garrettjoecox merged commit de430dc into HarbourMasters:develop Oct 27, 2023
8 checks passed
@Malkierian Malkierian deleted the fix-fire-temple-access branch October 27, 2023 01:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants