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

Companion conversations - Hit dice check for level capped players #535

Closed
1 task done
DarthParametric opened this issue Dec 2, 2021 · 2 comments
Closed
1 task done
Labels
Minor Issue Aesthetic issue or other problem that isn't really a bug Type: Scripting This issue is related specifically to a scripting bug/error

Comments

@DarthParametric
Copy link
Contributor

DarthParametric commented Dec 2, 2021

1.9.0 beta test observations:

  • Most of the companion conversations are gated by the player needing to have gained a level since the previous conversation, but this can prevent access to later branches if the particular companion was recruited or not spoken to until late in the game. The relevant starting conditionals probably need the addition of a check for the player already being level 20, or being on Lehon.
@DarthParametric DarthParametric added Minor Issue Aesthetic issue or other problem that isn't really a bug Type: Scripting This issue is related specifically to a scripting bug/error labels Dec 2, 2021
@DarthParametric
Copy link
Contributor Author

I have previously adjusted the final conversation conditional for Canderous - https://github.com/DarthParametric/K1_Canderous_Final_Conversation_Timing_Adjustment

@DarthParametric
Copy link
Contributor Author

DarthParametric commented Dec 5, 2023

1.10.0 beta test observations:

  • Doesn't seem like this is actually unlocking his final conversation. After coming back from the Bastila confrontation on the Temple Summit, his associated quest is closed as if you'd spoken to him. Edit: Seems like this is closed by the DLG for the beach showdown in both routes, but Canderous only says his 2¢ in the DS branch. Not sure it's really practical to change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Minor Issue Aesthetic issue or other problem that isn't really a bug Type: Scripting This issue is related specifically to a scripting bug/error
Projects
None yet
Development

No branches or pull requests

1 participant