Skip to content

Fix issue spellid #2

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

Open
wants to merge 23 commits into
base: master
Choose a base branch
from
Open

Fix issue spellid #2

wants to merge 23 commits into from

Conversation

wbaileydev
Copy link

No description provided.

BarbzYHOOL and others added 23 commits September 19, 2018 19:21
removing ScriptPCH , repair build using new headers
- Adds a `LearnAllOnFirstLogin` (default: false) option to teach the
player all the spells when he first logs in,
- Disabled by default because of performance concerns,
- Slight optimizations have been added to mitigate this issue
- However enabling this on a more populous server could cause issues
due to the current design of the module
- Anyone can add the missing spells directly in the code

- A bit hacky, but will do the job for now.
- Before that commit, .character level 80 didn't teach any spell
- Add cosmetic mage spells to the ignore list as some are not blizzlike and some are obtained ingame, thus cutting the reward from obtaining it and not being able to show it off
- Same for the warlock spell ritual of doom to summon doom guard (there is another spell anyway which is learned and is much easier to use)
feat(CI): move from Travis to GitHub Actions
Update the way the struct is called to correspond with current C++
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.

9 participants