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

fix(hook): OnBeforeInitTalentForLevel #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kaytotes
Copy link

This is a sister PR to azerothcore/azerothcore-wotlk#7389 to fix the usage of this hook within this module as it's used for testing builds.

Prior to this the hook wasn't fired at all so probably dead code here but might as well make it fit.

@FrancescoBorzi
Copy link
Contributor

cc @Yehonal

@kaytotes
Copy link
Author

Looks like we've got a bit of a conflict with the CI here and on azerothcore-wotlk and we'll have to merge one or the other in first to resolve it since this is pulling master from there to run the clang build.

@Helias
Copy link
Member

Helias commented Aug 18, 2021

let's merge the on in azerothcore-wotlk and later this

@Helias Helias changed the title Fix OnBeforeInitTalentForLevel fix: OnBeforeInitTalentForLevel Aug 18, 2021
@Helias Helias changed the title fix: OnBeforeInitTalentForLevel fix(hook): OnBeforeInitTalentForLevel Aug 18, 2021
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.

4 participants