Skip to content

Adding in 2 missing locations#2525

Open
dotzo wants to merge 3 commits intoOoTRandomizer:Devfrom
dotzo:two-missing-locations
Open

Adding in 2 missing locations#2525
dotzo wants to merge 3 commits intoOoTRandomizer:Devfrom
dotzo:two-missing-locations

Conversation

@dotzo
Copy link

@dotzo dotzo commented Jan 31, 2026

While doing work on Dev-Rob, I noticed 2 locations were missing from main dev that ought to be here: the shadow trial MQ wonder item, and a spawnable fairy collectible from MQ jabu. Adding these in just as they were over there.

@fenhl fenhl added Type: Bug Something isn't working Component: Logic Non-trivial changes to the JSON logic files Status: Needs Review Someone should be looking at it Component: Randomizer Core Generally the core functions of the python Status: Needs Testing Probably should be tested labels Jan 31, 2026
@r0bd0g
Copy link

r0bd0g commented Jan 31, 2026

If we're adding in even the missing fairy locations from my logic update from last year, there's also a gossip stone fairy missing outside deku. It won't affect main branch though b/c it's only relevant in mixed pools, but the logic is supposed to support mixed regardless.

@r0bd0g
Copy link

r0bd0g commented Jan 31, 2026

And I guess this is semi-related, but the lower lizalfos freestanding in DC is "missing" in MQ b/c it shares a location name with vanilla. Locations aren't supposed to share names between the two versions of the dungeon, and this location is the only one to do that at the moment. I just don't know if this counts as a location that's missing or not lol.

@dotzo
Copy link
Author

dotzo commented Feb 1, 2026

I didn't do a thorough check, to be fair. I was just working on fairy shuffle on realrob's branch and he has a new unit-test that pointed these out.

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

Labels

Component: Logic Non-trivial changes to the JSON logic files Component: Randomizer Core Generally the core functions of the python Status: Needs Review Someone should be looking at it Status: Needs Testing Probably should be tested Type: Bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants