Great guide!
You forgot to set charge usable outside of combat and only when 8-15yd away from target
Also Execute needs some extra rage to be used that's why you should put execute used only with 50+ rage. Sametime i would set mortar strike not usable with under 20%hp
If you want to pull spell names right from game client easily just go on bot interface tools -> fight class editor so it is hooked into client
And some other stuff like that