search

vikigenius / SpellbookMerge

SpellbookMerge is a mod for Pathfinder Wrath of the righteous that allows additional spellbook merging options
MIT License
19 stars 6 forks source link

EE Release with lots of new spellbooks #23

Closed wyakpara closed 2 years ago

wyakpara commented 2 years ago

This release aims to add all full caster spellbooks to every mythic path with spellbook merge enabled, along with a few other spellbook combos, including:

vikigenius commented 2 years ago

I suggest you remove any non full casters eg: Bard and Warpriest spellbooks from Angel and Lich respectively.

Because doing it will prevent you from getting higher level Angel and Lich spells.

wyakpara commented 2 years ago

Ohhh I didn’t even think about that. Tested angel with warpriest but wasn’t paying enough attention obviously. I’ll make the change and recommit a bit later. Thanks!

wyakpara commented 2 years ago

Warpriest and Bard removed from Angel and Lich, respectively.

vikigenius commented 2 years ago

Also, have you tested all of the functionality you added with the EE version? I don't have the game installed anymore so won't be able to test anything.

Also just as a sanity check, can you test if Angel is able to merge with Paladin and Paladin can get spells higher than 4th level (at higher levels ofc)?

wyakpara commented 2 years ago

Definitely tested with EE. I can test with Pally shortly.

wyakpara commented 2 years ago

Ugh. Well. Computer issues currently. Stay tuned; I'll double check Paladin once I have things settled and have a moment.

wyakpara commented 2 years ago

OK - confirmed, you can indeed get to 10th level spells with Pally merged with Angel.

Aside - to get other partial spellcasters to work with Angel/Lich, presumably you'd need to extend them the same way you do with Pally? I might try my hand at it later.

vikigenius commented 2 years ago

Yes, but the problem is that I have already extended some of them but only to some extend (eg: level 7 for Magus variants), redoing them to 9/10 would cause unintended results for earlier saves and mess with progressions of other merges and also your companions.

vikigenius commented 2 years ago

But thanks for the PR and testing, I will build locally and update the releases and mod version soon, and give you a shoutout for the contribution

vikigenius commented 2 years ago

For posterity, there are a few issues. Eldritch Scion was added but it does not have it's progression updated. This will be a problem since it won't get 7th level spells.

vikigenius commented 2 years ago

Actually never mind, it uses the Bard table which is updated already, so should not cause any issues.

Mythalar commented 2 years ago

He there,

I tested quite some of the new options, just one question : Since the spell tables are the sames, could Skald be added to the mythic paths that can merge with Bards?

vikigenius commented 2 years ago

It's already there for almost all. Only Aeon is missing, I will try to add at some point in the future. You are welcome to create a PR as long as you have tested it.