-
Notifications
You must be signed in to change notification settings - Fork 84
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] Add Gibdo Trade Sequence Options #917
base: develop
Are you sure you want to change the base?
[Enhancement] Add Gibdo Trade Sequence Options #917
Conversation
b18ba05
to
80cd83c
Compare
283faf5
to
3503f46
Compare
fd531ba
to
b85f7a7
Compare
b85f7a7
to
64d4960
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Solid job! Sorry it's taken a minute to get you some feedback.
So for the most part you nailed it with using GameInteractor_Should
in places to branch into custom behavior, with the main exception that you should introduce that behavior outside of the original source.
So instead of
if (GameInteractor_Should(VB_..., true)) {
// Original Stuff
} else {
// Custom Stuff
}
In your file, where you set *should = false;
is where the custom stuff should belong.
The other part is I'd like to not add new values to the EnTalkGibudRequestedItemIndex
enum and sRequestedItemTable
. Let me know if you run into trouble with this and I can try and figure something out.
When you get the chance, please review revised commit |
Add options to the Gibdo trade sequence beneath the well in Ikana.
Build Artifacts