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

Artifact Power Items support #114

Closed
wants to merge 2 commits into from
Closed

Conversation

Tarkumi
Copy link
Contributor

@Tarkumi Tarkumi commented Oct 20, 2016

No description provided.

@ckaotik
Copy link
Contributor

ckaotik commented Oct 22, 2016

Please note that this does not yet work for non-English locales. WowHead has a list of the spells (deDE locale in this case). Best would probably be to pick one of these spell ids and compare against the name returned by GetSpellInfo().

@tekkub
Copy link
Member

tekkub commented Oct 23, 2016

Awesome addition. I've merged it with what should be a more robust localization.

@tekkub tekkub closed this Oct 23, 2016
@ckaotik
Copy link
Contributor

ckaotik commented Oct 23, 2016

Awesome! However I think you got the wrong translation. There seems to be more than just the artifact empowering effects named "Empowering", that's why I linked the German list :) "
Macht verleihen" would be the correct translation in this case. I think this probably also affects other locales.

@tekkub
Copy link
Member

tekkub commented Oct 23, 2016

@ckaotik I get the spell name in 7913a4b, so it should work on any locale.

@ckaotik
Copy link
Contributor

ckaotik commented Oct 23, 2016

@tekkub Oh, nevermind, I must have been looking at the PR code instead of the actual commit. My bad, sorry!

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.

None yet

3 participants