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

RANDO: Fix logic with OGC and mixed entrance pools #2830

Merged
merged 4 commits into from
May 10, 2023

Conversation

leggettc18
Copy link
Contributor

@leggettc18 leggettc18 commented May 4, 2023

Adds a GANONS_CASTLE_LEDGE area and a BuiltRainbowBridge logic var. Added an event to GANONS_CASTLE_GROUNDS that sets BuiltRainbowBridge to true. Adds Entrances for GANONS_CASTLE_LEDGE to GANONS_CASTLE_GROUNDS when IsAdult and BuiltRainbowBridge (or the glitched logic that lets you skip the rainbow bridge) are true, HYRULE_CASTLE_GROUNDS when IsChild is true, and GANONS_CASTLE_ENTRYWAY When IsAdult is true (if child enters GANONS_CASTLE_LEDGE in game they are in HYRULE_CASTLE_GROUNDS and thus cannot go back into GANONS_CASTLE_ENTRYWAY). Adds Entrance for GANONS_CASTLE_GROUNDS to GANONS_CASTLE_LEDGE when BuiltRainbowBridge is true, or when the glitched logic that lets you skip the Rainbow Bridge is true.

If I'm understanding this correctly, this is also an issue upstream: gamestabled/OoT3D_Randomizer#673

Build Artifacts

Adds a `GANONS_CASTLE_LEDGE` area and a `BuiltRainbowBridge` logic var. Added an event to `GANONS_CASTLE_GROUNDS` that sets `BuiltRainbowBridge` to true. Adds Entrances for `GANONS_CASTLE_LEDGE` to `GANONS_CASTLE_GROUNDS` when `IsAdult` and `BuiltRainbowBridge` (or the glitched logic that lets you skip the rainbow bridge) are true, `HYRULE_CASTLE_GROUNDS` when `IsChild` is true, and `GANONS_CASTLE_ENTRYWAY` When IsAdult is true (if child enters `GANONS_CASTLE_LEDGE` in game they are in `HYRULE_CASTLE_GROUNDS` and thus cannot go back into GANONS_CASTLE_ENTRYWAY). Adds Entrance for `GANONS_CASTLE_GROUNDS` to `GANONS_CASTLE_LEDGE` when `BuiltRainbowBridge` is true, or when the glitched logic that lets you skip the Rainbow Bridge is true.
@leggettc18
Copy link
Contributor Author

This should fix issue #2810

@leggettc18 leggettc18 linked an issue May 4, 2023 that may be closed by this pull request
Copy link
Contributor

@Archez Archez left a comment

Choose a reason for hiding this comment

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

LGTM

@leggettc18
Copy link
Contributor Author

Waiting on the 3drando version to get merged so I can send any changes they ask of me over here as well.

@leggettc18 leggettc18 merged commit a72997f into HarbourMasters:develop-spock May 10, 2023
7 checks passed
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.

Song of Time unobtainable in seed
4 participants