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

Hlam dialog fix by sanpats #26

Merged
merged 6 commits into from
Sep 21, 2018
Merged

Hlam dialog fix by sanpats #26

merged 6 commits into from
Sep 21, 2018

Conversation

sanpats
Copy link
Contributor

@sanpats sanpats commented Sep 21, 2018

Modified the fix by NinjaSpectre, but change nkhelgarismonk=1 to nkhelgardecided=1 when he decided to stay as a fighter, to ensure compatibility with later dialogs that may think Khelgar is a monk when he stay as a fighter. In case that Khelgar decided to be a monk, I set both nkhelgarismonk and nkhelgardecide to 1. All condition checks in this dialog that check nkhelgarismonk was changed to check nkhelgardecided instead.

Merge from the main branch
Neverneath end quest fix
Temple of Seasons tweak by kevL
Nevalle dialog fix by kevL
Modified the fix by NinjaSpectre, but change nkhelgarismonk=1 to nkhelgardecided=1 when he decided to stay as a fighter, to ensure compatibility with later dialogs that may think Khelgar is a monk when he stay as a fighter. In case that Khelgar decided to be a monk, I set both nkhelgarismonk and nkhelgardecide to 1. All condition checks in this dialog that check nkhelgarismonk was changed to check nkhelgardecided instead.
Modified the fix by NinjaSpectre, but change nkhelgarismonk=1 to nkhelgardecided=1 when he decided to stay as a fighter, to ensure compatibility with later dialogs that may think Khelgar is a monk when he stay as a fighter. In case that Khelgar decided to be a monk, I set both nkhelgarismonk and nkhelgardecide to 1. All condition checks in this dialog that check nkhelgarismonk was changed to check nkhelgardecided instead.
@kevL kevL merged commit ef31976 into nwn2fixes:master Sep 21, 2018
@kevL
Copy link
Collaborator

kevL commented Sep 21, 2018

okie doke - I think this is what i was concerned about in the previous version. tks

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