[FEATURE] Allow Felstorm Demo spell be in the spell priority list
marekCs opened this issue ยท 5 comments
Before You Begin
- I confirm that I have downloaded the latest version of the addon.
- I am not playing on a private server.
- I checked for an existing, open ticket for this request and was not able to find one.
- I edited the title of this feature request (above) so that it describes the issue I am reporting.
Feature Request
Since the Demo pet very often triggers Felstorm out of the blue, wouldn't it be a good idea to place the Felstorm spell as part of the priority list (and on the pet's tab switched off by default) and only use it as a recommendation when the demonic pet is within the melee range of the mob/adds?
Additional Information
No response
Contact Information
No response
I haven't seen any recent material about folks manually controlling Felstorm, but it's something I could do. One advantage would be that you'd also get more control over Guillotine. I'll think about it a bit.
Hello, I fully respect that you do not have much free time to add this action. I would just ask for a quick review pls, is it ok like this?
felstorm = {
id = 89751,
cast = 0,
cooldown = 20,
gcd = "spell",
school = "physical",
spend = 60,
spendType = "energy",
startsCombat = false,
toggle = "cooldowns",
handler = function ()
applyBuff( "felstorm" )
end,
}
I'm just not sure if school is nature or physical and also if it's 60 energy, can I write spend = 60