Skip to content
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

Optional Rogue Stealth Ranks - Handbooks #350

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Grimfeather
Copy link
Contributor

@Grimfeather Grimfeather commented Aug 27, 2024

Related to: #194 and #213

I've tried in several ways to add new stealth ranks directly to the rogue trainer spell lists.
But it simply won't work. So the next best thing I could think of are these handbooks.

  • create 4 new stealth spells

  • create 4 new handbooks

  • add 4 stealth handbooks to all rogue trainer vendor lists.

  • add trade option to all rogue trainers

  • still to do: Let higher ranks overwrite lower ranks of Stealth. ( right now you end up with 4 separate Stealth buttons. )

This does require an update to spell.dbc and patch-V
I added item.dbc to patch-V and updated the spell.dbc in patch-V as well.
I uploaded them to mediafire, they can be downloaded here:
https://www.mediafire.com/file/ft5j96nkkt6xp6n/Spell.dbc_%252B_Patch-V.7z/file

If someone knows how to make the higher stealth ranks replace the lower ranks, please let me know.
I thought spell_ranks does this, but I think the client needs to get the spell_ranks information as well, it's a display error after all.
Or a script that removes the previous Stealth rank before learning the new one.

- create 4 new stealth spells
- add 4 stealth handbooks to all rogue trainers.
- add trade option to all rogue trainers

- still to do: Let higher ranks overwrite lower rank of Stealth.
( right now you end up with 4 separate Stealth buttons. )
…h_ranks.sql

Add new stealth spells to spell_ranks
…e_rogue_stealth_ranks.sql

removed an update to gossip_menu_option that wasn't needed and causes an issue.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant