Check if you did Sha this week Macro

[rotgut]

Keep forgetting if you did Sha this week or not?
Here's your solution:

/run local z,t,s={[32099]='Sha of Anger this week.'},GetQuestsCompleted();for c,v in pairs(z) do if t[c] then s='' else s=' not' end print('You have'..s,'done',v) end

Enjoy

[Deleted]

While I'll commend you for your effort (especially in making it expandable, I like it!), your macro can still be improved. Here's how I'd make it look:

/run local z,t={[32099]='Sha of Anger this week'},GetQuestsCompleted()for c,v in pairs(z)do print(("You have%s done %s."):format(t[c] and " not" or "",v))end

[Alahondor]

While I'll commend you for your effort (especially in making it expandable, I like it!), your macro can still be improved. Here's how I'd make it look:

/run local z,t={[32099]='Sha of Anger this week'},GetQuestsCompleted()for c,v in pairs(z)do print(("You have%s done %s."):format(t[c] and " not" or "",v))end

Your macro lies to me and tells me that I have already done it, when I haven't.

[Alvito]

Just ctrl v this into a macro and you can see if you have done Sha or Galleon that week.

/run local c = IsQuestFlaggedCompleted(32099)print("You have "..(c and "" or "not").."done Sha of Anger this week..")
/run print("You have "..(GetQuestsCompleted()[32098]and "" or "not").." done Galleon this week.")

Might as well dump the 2 in together.

[Deleted]

Yeah, I had messed the order up initially. That's what I get for typing too quickly.

/run local z,t={[32099]='Sha of Anger this week',[32098]='Galleon this week'},GetQuestsCompleted()for c,v in pairs(z)do print(("You have%s done %s."):format(t[c] and "" or " not",v))end

[Venara]

Or http://www.curse.com/addons/wow/world-boss-status ;-)

[Evil Inside]

Or http://www.curse.com/addons/wow/world-boss-status ;-)

I did not know this exists. Thank you!