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

[Enhancement] Receive oceanside spider house wallet reward on any day #861

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

lscholte
Copy link
Contributor

@lscholte lscholte commented Nov 16, 2024

The wallet reward for clearing the oceanside spider house is normally only obtainable on the first day. This enhancement will allow the wallet reward to be collected on any day.

I'm also not sure if this would better in a different section. Perhaps restorations because this enhancement does recreate MM3D behaviour?

Build Artifacts

@lscholte lscholte force-pushed the wallet-reward-any-day branch 2 times, most recently from 0e11640 to 31b3466 Compare November 16, 2024 17:43
@lscholte lscholte force-pushed the wallet-reward-any-day branch from 31b3466 to 4315c3b Compare November 16, 2024 17:46
@Eblo Eblo mentioned this pull request Jan 27, 2025
46 tasks
Copy link
Contributor

@Eblo Eblo left a comment

Choose a reason for hiding this comment

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

Same deal as your other PRs, needs hookified

I'm also not sure if this would better in a different section. Perhaps restorations because this enhancement does recreate MM3D behaviour?

"Time Cycle" makes sense to me.

switch (day) {
case 0: // first day
if (CHECK_WEEKEVENTREG(WEEKEVENTREG_RECEIVED_OCEANSIDE_WALLET_UPGRADE)) {
if (!CHECK_WEEKEVENTREG(WEEKEVENTREG_RECEIVED_OCEANSIDE_WALLET_UPGRADE) &&
Copy link
Contributor

Choose a reason for hiding this comment

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

Instead of directly using the CVar in the actor file, you should create a new VB, wrap the affected logic with a GameInteractor_Should call, and handle the custom logic in a new enhancement file. One example you could look at is your very own VB_DEKU_GUARD_SHOW_SEARCH_BALLS.

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.

2 participants