-
-
Notifications
You must be signed in to change notification settings - Fork 23
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
1.20.3 #880
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
## Bug fixes - Fix save file bloat from reading popups
Accidently merged into wrong branch so uhh yeah
to somewhat match base games's xmls even if they aren't always consistent there
## Improvements - Added ReuseDialogueOptionsListFrom, DisablePersistentCondition, and ParentIgnoreNotRevealed to their respective schemas - Specify (and also correct some) minOccurs & maxOccurs on EVERYTHING (this one is for you visual studio) - Rearranged things to better match the xmls from vanilla
needs to divide not multiply
So that we can parent it to a root detail
because it being at zero makes it use the sphereOfInfluence instead
even though this already happens in CharacterDialogueTree.EndConversation itself, I did it for Ixrec anyways.
## Improvements - fix #859: CharacterDialogueTree.InConversation is now checked in our attachment patch to avoid unnecessary EndConversation calls.
## Bug fixes - Fix disabled reference frames not being disabled (I swear I've done this 3 times now)
## Bug fixes - Fixed sectors being rotated on tidally locked bodies
## Minor features - Added `parentPath` to scatter. This parent should be at the position where you'd like to scatter (which would usually be zero). ## Bug fixes - Fix scatter not working properly on tidally locked bodies
## Bug fixes - Fixed returning to solar system radius not being calculated correctly
xen-42
approved these changes
Jun 4, 2024
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.
lgtm
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Minor features
parentPath
to scatter. This parent should be at the position where you'd like to scatter (which would usually be zero).Improvements
Bug fixes