TellMeWhen

TellMeWhen

24M Downloads

[CF 1579] "Stunned" Group

tmw-issue-import opened this issue ยท 4 comments

commented

Hey man, love tell me when just having a little issue:

Debuff group "Stunned" has binding shot in it (Hunter spell) but in BFA binding shot is now just a root (not a stun)

thanks <3


Posted by CurseForge user Quin69 | Imported from CurseForge issue #1579 | Raw

commented

The CC 'groups' are all out of date, as far as I'm aware.


Posted by CurseForge user Jalopyy

commented

Correct - I'll make this change and a few others as well (some shadow priest CC since I play one), but I don't have the time anymore to comb through every class spell looking for what minor changes have been made - maybe something ended up with a new spellID, or something does something different, or maybe WW monks have a new 1 second stun that needs to be added.

 

TMW has approx 400 spells in its equivalency lists. Its just too hard for me to try and keep them maintained. I put out a call for help in the changelog for 8.5.0, but got no input. I'll leave this issue open for a while to gather any other changes that people would like to contribute. PRs on https://github.com/ascott18/TellMeWhen are also welcome. https://github.com/ascott18/TellMeWhen/blob/master/Components/Core/Spells/Equivalencies.lua&nbsp;


Posted by CurseForge user Cybeloras

commented

I missed the call for help, but will try to go over some of the stuff in the next week or so. 

 

In this vein - what is your stance on NPC abilities? Current xpac? none at all?


Edited Sep 26, 2018

Posted by CurseForge user ariehg

commented

I generally don't add them simply because I don't invest the effort. I'd say if they're significant parts of boss fights, then they're worth adding. For example, if the boss applies a 20 second stun to the whole raid during a phase transition, then that isn't significant (because you can't do anything about that). However, if the boss applies an 8 second stun to the tank because you failed a taunt swap mechanic, then that's worth adding.

 

Back in Cata, there were equiv lists for each tier containing all the significant interruptible spells for that tier (

Tier11Interrupts = "_83703;_82752;_82636;_83070;_79710;_77896;_77569;_80734;_82411",
). I'd totally be open to bring that concept back, and expand it beyond just spell interrupts. Probably also one for 5 mans since Mythic+ is a thing now. We could have both this AND including things in their respective category (e.g. the stun goes in both Stunned and UldirDebuffs).


Posted by CurseForge user Cybeloras