Hekili Priority Helper

Hekili Priority Helper

44M Downloads

[FEATURE] Allow Felstorm Demo spell be in the spell priority list

marekCs opened this issue ยท 5 comments

commented

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

commented

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.

commented

Thank you very much.

commented

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,
    }
commented

I'm just not sure if school is nature or physical and also if it's 60 energy, can I write spend = 60

commented

The energy piece won't actually check the pet's resources. The school doesn't matter (but it's almost certainly not nature).