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

Dialogue edits and additions #588

Merged
merged 9 commits into from
Apr 10, 2024

Conversation

Dweblenod
Copy link
Member

A review of existing dialogues and the creation of new ones.

Some changes that are notable:

  • The consort cult dialogue is more interactable and can result in the player gaining a Long Forgotten Warhorn
  • Iguanas asking to eat your hair can now be given the ponytail item
  • Dialogues which referred to family members have been made more ambiguous
  • Dialogues where the response message was not the actual players words now use square brackets
  • Some lore related dialogues have been updated to reflect other upcoming content

All dialogue is written by Dweblenod

…esponses)

- Created new dialogues
- Created new item tag for magic weapons and added a getter method for ItemRightClickEffect in WeaponItem
- Added new constructor for GiveItem Trigger
@Dweblenod Dweblenod added the enhancement New feature or request label Apr 1, 2024
- generic_thanks now has flag for if the player in the conversation is the one who helped the consort
@Dweblenod Dweblenod marked this pull request as ready for review April 4, 2024 11:26
@Dweblenod
Copy link
Member Author

There is still some more dialogue I would consider adding and I will need to fix the merge conflicts but Im going to start marking this as ready for review

CHANGELOG.md Outdated Show resolved Hide resolved
Copy link
Member

@kirderf1 kirderf1 left a comment

Choose a reason for hiding this comment

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

Great!

- Added book_purpose dialogue
@Dweblenod
Copy link
Member Author

Added one more commit, and will consider this done if the new commit passes review!

@kirderf1 kirderf1 self-requested a review April 9, 2024 20:36
@Dweblenod
Copy link
Member Author

I see you self-requested a review, did you intend to review it more or should I take your approval as an opportunity to merge?

@kirderf1
Copy link
Member

It's fine to merge by me.

@Dweblenod Dweblenod merged commit efc001f into lunar-sway:quest_systems Apr 10, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants