Open
Conversation
Added spells to NPCs from: Thais, Carlin, Venore, Ab'dendriel, Kazordoon, Edron Enabled learning for all game spells Added new spells from Isle of Destiny I think this cover all game spells but if there is any important missing NPC let me know. I will add the rest of the NPCs from the other cities when I got the time, you can help if you want.
Collaborator
|
Maybe you can add new config to toggleLearnSpells in config lua so it enable/disable spell:needLearn(true), so we can disable it |
You may now enable or disable learning spells in config.lua
Contributor
Author
Sent new toggle at config, I just updated the PR |
This fix some issues when buying spell that use the same keyword combinations, like haste or light, etc.
Some more details fixed for this PR that incliude new beginer spells and a fix in the keyword order of puffels NPC.
…61315/crystalserver into feat-spell-learning-system
Added executeChain target check to avoid console error flood.
Some updateds in spells I was missing on. Sorry, It should be accurate with git now.
Fixed Rune Spell Names and Invisibility, also the keyword order in some more spells.
Fix order in keywords for a proper reaction of the NPC
…61315/crystalserver into feat-spell-learning-system
As the title says, just correcting duplicates with wrong names.
…61315/crystalserver into feat-spell-learning-system
More debug of spells system and a new NPC for party spells at Edron Academy.
I just knew Monks autolearn the virtue spells when promoted so I kinda fixed it here. Also fixed more spell keywords and added Eremo NPC spell learning system. Enpa Rudra now can promote monks and other players.
Added new basic paladin spell Lesser Ethereal Spear to NPCs and Dawnport Vocation Trial. Lesser Front Sweep has been added for knights in the Dawnport Vocation Trial aswell.
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Added spells to NPCs from: Thais, Carlin, Venore, Ab'dendriel, Kazordoon and Edron
Enabled learning for all game spells
Added new spells from Isle of Destiny
I think this cover all game spells but if there is any important missing NPC let me know. I will add the rest of the NPCs from the other cities when I got the time, you can help if you want.