Randomizer Location Exclusion List#2070
Conversation
- Adds Scrubs, Merchants, Shops. Adult Trade items, MQ Dungeons, and Bean Salesman - Fixes Kokiri Sword Chest, Weird Egg, and Frog Song Rupees from misbehaving - Updates Gerudo Fortress logic to be dependent on Open/Fast/Normal Fortress - Removes Invalid Check - Adds RO Enum for MQ settings - Updates magic values to RO enum
|
|
Great work here thanks for doing this! |
once this is taken care of i'll remove the do not merge label, mq cleanup can be done as a separate PR if you'd like |
Those 3 scrubs now show always in the exclusion list as per 467131e. I realized that I leaked some of my code for the Check Tracker in there. While not directly harmful, the RAND_INF flags being added aren't required for this code. This is resolved by e75f971. |
…up when Adult Trade Items was off
|
@sonoftunk is there anything else you're still wrapping up on this one or is it ready for a final look through? |
|
@briaguya-ai No more changes. Good for final review
Edit: Snipped automatic email attachment
|
… Merge conflict caused by this PR and another PR both adding MQ dungeon count enums.
briaguya0
left a comment
There was a problem hiding this comment.
couple tiny changes then this should be good to go
Location Exclusion List - Applies suggested dungeon item defaults Co-authored-by: briaguya <70942617+briaguya-ai@users.noreply.github.com>
Most fixes for #1620
Build Artifacts