-
Notifications
You must be signed in to change notification settings - Fork 65
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
Conversation
…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
- generic_thanks now has flag for if the player in the conversation is the one who helped the consort
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 |
# Conflicts: # CHANGELOG.md
… generic closing message used by it
- Expanded frog_creation dialogue - Changed the weights of 3 dialogue
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.
Great!
- Added book_purpose dialogue
Added one more commit, and will consider this done if the new commit passes review! |
...erated/resources/data/minestuck/minestuck/dialogue/consort/book_purpose/buy_my_products.json
Outdated
Show resolved
Hide resolved
src/main/java/com/mraof/minestuck/data/dialogue/ConsortDialogue.java
Outdated
Show resolved
Hide resolved
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? |
It's fine to merge by me. |
A review of existing dialogues and the creation of new ones.
Some changes that are notable:
All dialogue is written by Dweblenod