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

Fairysanity #4744

Open
wants to merge 52 commits into
base: develop
Choose a base branch
from
Open

Fairysanity #4744

wants to merge 52 commits into from

Conversation

Pepper0ni
Copy link
Contributor

@Pepper0ni Pepper0ni commented Dec 20, 2024

Built from @boomshroom 's work.

Shuffles all fairy fountain, song, gossip stone and bean sprout fairies. does not affect pot fairies (those are governed by Pot Shuffle) or butterfly fairies at this time. Wonder item fairys are also not randomised.

Build Artifacts

boomshroom and others added 30 commits September 3, 2024 22:12
This is necessary since, unlike chest or scrub grottos,
fairy fountains, lacking any elements that would normally
differ between grottos, often have identical respawn data.
This change enables fairy shuffle to correctly identify
which fairy fountain was entered so it could load the right
checks.
Logic might look a little weird. While the Gossip Stones
respond to Sun's Song, the scene transition takes priority,
so the fairies aren't actually obtainable. Adult does not have
day and night versions and is fully capable of acquiring the
fairies with Sun's Song.

Song of Time is similar, but rather than being dependant on
age, it's dependant on the timesaver to switch ages anywhere
with Song of Time. In this case, the fairies still spawn,
but the age transition deletes them before giving the chance
to collect them. While I could've chosen to ignore this and
simply require the player to disable the enhancement before
playing if it's their only option, I instead decided to
accomodate the enhancement and require one of the other songs
in order for the check to be in logic.
Somehow, it was previously possible for Navi to get detected
as a randomized fairy and rendered as an item. I was not
able to reproduce the behavior, but this should prevent it
from happening again.
@Pepper0ni Pepper0ni marked this pull request as ready for review December 24, 2024 21:21
@Pepper0ni Pepper0ni changed the title Fairysanity WIP Fairysanity Dec 24, 2024
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.

3 participants