A better way to cast things is not using ActionList:Cast().. We already have things behind the scenes that tell you if a skill is ready to cast, which would alleviate a lot of conditions for certain things, and also be more efficient because you would not need to put manual delays in after the skill is cast. It would also help to general profiles as if you don't have the skill, .isready will always be false anyway. You would not require after cast delays using this method, which means you would never had delay problems.
This code will never cast unless the skill is actually castable, so no need for conditions or timers since tricks of the trade is a proc skill, when .isready = true its castable.
.isready incorporates the games ACTUAL cancast, which basically means if the skill is greyed out on your hotbar .isready will be false.
You should never use ActionList:Cast() please refer to this
https://github.com/MMOMinion/FFXIVMinion/wiki/Action
This same principle applies to the way inventory items are used as well
https://github.com/MMOMinion/FFXIVMinion/wiki/Item
There is no need to track cooldowns and such.
Code:
--Tricks of the trade
--using :Get(id) updates and returns the action, if you do not use :Get(id) the data in the action wont be updated
local action= ActionList:Get(100098)
if ( action.isready ) then
myspell:Cast()
end
This code will never cast unless the skill is actually castable, so no need for conditions or timers since tricks of the trade is a proc skill, when .isready = true its castable.
.isready incorporates the games ACTUAL cancast, which basically means if the skill is greyed out on your hotbar .isready will be false.
You should never use ActionList:Cast() please refer to this
https://github.com/MMOMinion/FFXIVMinion/wiki/Action
This same principle applies to the way inventory items are used as well
https://github.com/MMOMinion/FFXIVMinion/wiki/Item
There is no need to track cooldowns and such.