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

Expand scarecrow behavior to three options #2346

Open
wants to merge 2 commits into
base: Dev
Choose a base branch
from

Conversation

GSKirox
Copy link
Collaborator

@GSKirox GSKirox commented Jan 8, 2025

Change the free_scarecrow option to scarecrow_behavior with 3 options :

  • Vanilla
  • Fast : you don't need to play the song to summon Pierre, just to pull an Ocarina, but you still need to set a song to child and adult Bonooru.
    This is logically identical to Vanilla.
  • Free : same behavior as the old free_scarecow option.

@GSKirox GSKirox added Type: Enhancement New feature or request Status: Needs Review Someone should be looking at it Status: Under Consideration Developers are considering whether to accept or decline the feature described Component: Patching Affects the patching of the ROM labels Jan 8, 2025
@fenhl fenhl added the Component: Setting specific to setting(s) label Jan 8, 2025
@fenhl fenhl removed the Status: Needs Review Someone should be looking at it label Jan 8, 2025
@cjohnson57 cjohnson57 added Status: Waiting for Author Changes or response requested and removed Status: Under Consideration Developers are considering whether to accept or decline the feature described labels Jan 13, 2025
@r0bd0g
Copy link

r0bd0g commented Jan 14, 2025

I guess don't forget to make this work together with #2221 when that comes in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Patching Affects the patching of the ROM Component: Setting specific to setting(s) Status: Waiting for Author Changes or response requested Type: Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants