search

valkyrnstudios / RankSentinel

MIT License
4 stars 2 forks source link

Scourge Strike Doesn't Have Rank 7 #59

Closed Gogo1951 closed 2 years ago

Gogo1951 commented 2 years ago

image

image

Only went to 6 in the data, but the add-on thinks there's a Rank 7.

https://www.wowhead.com/wotlk/spell=49921/plague-strike

Gogo1951 commented 2 years ago

image

Gogo1951 commented 2 years ago

Some how this got included in the Plague Strike group. image

Gogo1951 commented 2 years ago

This group of Glyphed Death Coil seems to have gotten left out, but one of the spells incorrectly got added to the Plague Strike group. May be worth checking the groupings, but that's the only one I found that was odd for DKs.

image

SabreValkyrn commented 2 years ago

That is not the right GSheet, per README. https://docs.google.com/spreadsheets/d/1jtx1WyfChzACzh0WBWANtrqkRtS3D-zPWqs3eOnyVvY/edit

image

But that did indeed get lumped into { 45462, 49917, 49918, 49919, 49920, 49921, 62904 }, Plague Strike which is confusing.

SabreValkyrn commented 2 years ago

Probably fixed by this change.

image

v2.0.5c