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

Priest racial abilities do not show up in the training list. #12

Open
davidcraig opened this issue Jun 29, 2019 · 1 comment
Open

Priest racial abilities do not show up in the training list. #12

davidcraig opened this issue Jun 29, 2019 · 1 comment
Assignees
Labels
good first issue Good for newcomers Priest
Milestone

Comments

@davidcraig
Copy link
Collaborator

Spells such as Elune's Grace, Shadowguard, Fear Ward, etc are not currently accounted for in the training list.

@davidcraig davidcraig self-assigned this Jun 29, 2019
@davidcraig davidcraig added this to the v1.1.0 milestone Jun 29, 2019
@davidcraig davidcraig added the good first issue Good for newcomers label Oct 20, 2019
@davidcraig
Copy link
Collaborator Author

davidcraig commented Oct 20, 2019

As an explanation of how this will be achieved, the table is laid out like this

local PriestRacialClassSpells = {
  ['dwarf'] = {
    [10] = {
      ['desperate_prayer:1'] = {
        name = 'Desperate Prayer',
        rank = 1,
        id = 13908,
        level = 10,
        cost = 'quest'
      }
    },
   ...

So each race has a key, we can change the keys if it's easier to match them against a function from wow which retrieves the player race. But we simply just need to add the abilities into the main loop based on player race.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers Priest
Projects
None yet
Development

No branches or pull requests

1 participant