-
Notifications
You must be signed in to change notification settings - Fork 60
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
Mod Compat PR #501
Mod Compat PR #501
Conversation
FORGET TO REVERT THE TEST CHANGE YOU MADE TO BRISTLE MEAD
… breaks. Also, spatial storage is explicitly designed to be able to move budding blocks. Don't remove that from AE2. Resonance exists anyways, its not like this breaks things too much...
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.
Lots and lots of good stuff!
Left you a few notes.
@@ -1,7 +1,6 @@ | |||
{ | |||
"replace": false, | |||
"values": [ | |||
"#spectrum:budding_blocks", |
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.
This entry here mainly is:
- to give players a reason to go back to the DD from time to time
- make them not complain when they notice that moved budding blocks can not be used to generate ink, once the ink update it out (which will still take a while and will prob. be 1.21+, so eh)
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.
doesn't the resonance enchantment have these same issues anyways?
"immersive_weathering:spectrum/purple_bark": "Oak Bark?", | ||
"immersive_weathering:spectrum/blue_bark": "Oak Bark?", | ||
"immersive_weathering:spectrum/green_bark": "Oak Bark?", | ||
"immersive_weathering:spectrum/red_bark": "Oak Bark?" |
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.
The other spectrum blocks do use scrambled names instead.
Why should we handle it differently here?
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.
Unlike most spectrum blocks, this is something that a player is VERY likely to encounter in natural gameplay, as stripping a colored tree will yield its respective bark regardless of progression. Additionally, my planned cloak overhaul PR (that is waiting on changes to Revelationary) will be adding more instances like this.
I am completely burnt out right now so there's still some TODO files for what i was going to do later, also there is no guidebook entries.
Let this go to main?
Can't be bothered to write out everything in this PR, also i changed kindling AI a bit while working on the vivisection recipe
yes there is a merge conflict but right now i don't care