Achievement: To All the Squirrels I Once Caressed?

Re: Issues with this guide…Like TwoToad said, he appreciated any feedback and edits.

Issues with this guide…I have a number of issues with this guide.The first and most important is the typo in the guide’s title…WoWPro.Achievements:RegisterGuide(‘TwoToadSqu’,’MoP Zones’, ‘To All the Squirrels I Once Carressed?’, ‘Achievement’, ‘Mists of Pandaria’, ‘TwoToad’,’Neutral’, function()Secondly, do we really need a note telling us the title of the guide?And lastly, most of the notes in the steps are just overstating the obvious. We do not need a note to tell us to love the animal named in that step. The notes giving specific details are all that is needed, such as the note for the dancing water skimmer.Don’t get me wrong. I appreciate the effort put into writing this guide. Just blunt constructive critisism.EDIT: I have made some edits in the guide to reflect these comments.

TargetThis should work the same using what i wrote, your just putting the target first.  I only called the value emote, cause that would be the most common use for it.  You could easily label the field macro (if it wasn’t a resevere word)C A Blade Fit For A Champion|QID|13666|M|60.4,52.0|Z|Grizzly Hills|T|Lake Frog,use Warts-B-Gone Lip Balmn/kiss|L|44981|N|Kiss frogs till you get a princess. Then ask for the blade.|Another solution would be since you only use the balm once and it has a 30 min cooldown, set it up with a Use statement and a |S| tag.FYI: In case you missed it, I updated the main GITHUB site by mistake when I was attempting to update my fork.  Really sorry and wasn’t my intention on forcing this issue to the main.  I am really flexable on this, I just feel its putting alot on the gudes themselves and works for this one circumstance although there maybe a few others (like the achievement for possible more than one target in the Klaxxi daily).  But this would work as well by doing:|T|MonsterA,target Monster Bn/target Monster C…etcI would rather leave the target to just target things.  Maybe create another for macros so that we have target, use, and marco?

re: Tangential question – n?BINGO!Yeah, the n stands for a new line.  It is used for multiline macros.  Somtimes ‘;’ work, but n’s always work.

Tangential question – n?Ive seen the “n” before… but I don’t know what it does? from this context it is like a <CR> so you are doing multiple commands in one line?

Re: TargetBecause then I can do this too:C A Blade Fit For A Champion|QID|13666|M|60.4,52.0|Z|Grizzly Hills|T|/use Warts-B-Gone Lip Balmn/target Lake Frogn/kissn|L|44981|N|Kiss frogs till you get a princess.  Then ask for the blade.|I think the cleartarget is used in the addon just to force the target command to not leave you with your current target, as it wil not change the target if the object is not found.  

TargetThats’s alot of extra steps that you are requiring the guides to control.  The /clear target  is unecessacry in this instance since it changes when you find the target any way.  But you can add it in if you wish.  So why not just have: this?C Rams on the Lam|QID|25905|M|69.76,57.21|N|Get within 15 yards of them.|NC|T|Stolen Ram,whistle|

Re: TargetI think i disagree with your approach a bit.   How is this?|T|XXX|:  Does the traditional  /cleartargetn/targetexact XXXn/run if not GetRaidTargetIndex(‘target’) == 8 and not UnitIsDead(‘target’) then SetRaidTarget(‘target’, 8) end|T|XXX: Does exactly what the string says.So in Dun Morogh, “Lambs on the Lam” I can do:C Rams on the Lam|QID|25905|M|69.76,57.21|N|Get within 15 yards of them.|NC|T|/cleartargetn/target Stolen Ramn/whistle| 

ugh – frog kissingI hated that daily, it was sooooo far away, and I would count sometimes and be well over 50 frogs before I found the girl (or until someone else showed up and found it on thier first /kiss).  I took 4 char (or maybe 5… I liked heirlooms) to exalted with argent crusade and that was by far the worst part of it.  (altho I was quite sick of joisting too)

EditingGreat!!! I can edit now.  Thanks.  Yes, there are a bunch of places where this feature will come in handy. The good thing about it is that you can just fly over and zappp it without even seeing it.  Makes it so much easier to find things.

Re: TargetCool!   Will pull in.
BTW, please try to edit a guide. I reset your settings and you should be able to now!
 
This will be useful for some dailies, that require you to target and kiss frogs and but anti-wart lotion on.

TargetThe problem is with the target function itself.  It’s desgned just for NPC’s and not for critters.  However, I updated my fork on GITHUB to not only handle the targetting problem on critters, but also does the emote along with it.  I still can’t edit the guides here on the site, so you will have to pull it off my fork.  It required changing the parser and this guide. guide-commit d440d15c7f3323671bbba55b7b25ef34bebb89f6parser-commit 840849f8fbd0f4ea14de74e4177e5fc96e1eca3b

Re: Golden water skimmerI added a comment into the GL guide. I put it right at the beginning since I’m not sure at what point you become phased wrong.  If any one wanted to test that… feel free to move it 🙂

Re: Golden water skimmerYeah, the Golden Lotus guide is perfect, as you would not be phased out before starting it.A note in the achievement guide would be good as well.

Golden water skimmerSince it (the critter)  is in Vale of Eternal Blossoms, there is no leveling guide to put it it… maybe a note in the beginning of the Golden Lotus guide?

Re: I think the Z tag errors areBlech.   That one may be my fault.   I’ll look into it before the next update.The zone checking logic needs a little TLC.

I think the Z tag errors areI think the Z tag errors are from the capital T in Valley of The Four Winds, I went and looked at another guide and its Valley of the Four Winds there. :/eddted to add: I changed the first instance of Valley of The Four Winds, to use the lowercase “t” and now I am certain that is why you are getting the Z tag errors. Also, the tom tom arrow is updating with each step as it is supposed to.Yet another edit.  After I finished the guide and got the achievement… I got an error that said: “WowPro:Guide TwoToadSqu has an bad QID! [false]” ( i dont see from a quick look thru why it is saying that)Last, IMHO (or not so humble) the order could be improved. You are flying thru Vot4Winds 3 differant times.  I usually try to plot out the locations on a map and arrange them according to the “picture” it shows me.

Thanks Emmaleah! I updatedThanks Emmaleah! I updated the guide with the QIDs now. The arrow is still not updating though. I also noticed WoW-Pro is spitting out errors in the chat window concerning the |Z| tags for the Valley of the Four Winds. Someone else had this issue as well, did this ever get fixed?

To get the QID for achievements…see this post.https://wow-pro.com/node/3448#comment-40995or… if background info isn’t interesting to you, just type “/run WoWPro.Achievements:DumpInfo(6350)” in your chatbox in game. It will put some output in the center of your screen that among other things shows the proper QID.The tomtom arrow may update properly with the QID in there.I like Ludo’s suggestion for making the |U| accept an emote, if that doesn’t work for whatever reason, I was thinking a simple solution is to tell them at the beginning of the guide to make a macro and put it on your action bar somewhere.

The targetting did seem toThe targetting did seem to work ok when I tested it, it was just a concern because I had tried “/tar Mongoose” manually before and had gotten the Pup instead.I couldn’t figure out what the |QID| should be for this achievement and how to look it up. The Explorer achiev used one… And I believe I used the |ACH| tag correctly (it just took me a few minutes to figure it out).What about the problem of TomTom not updating? I target a Mongoose, manually type /love, the guide goes to the next part of the achievement, but the TomTom arrow stays pointing at the area for the Mongoose. I had to go into the Current Guide, then re-check the step I just completed (it already had a check, I just clicked it again) and then the arrow updated.

re: Oh, also I was thinking…We have added things like finding lost tresures into the other guides, where it flows naturally.If the player is in an area where the right creature is present, this is an ideal |RANK|3|  item! 

Re: A few issuesThe name you pass to the |T| tag gets passed to a macro that looks like:
[code]
/cleartarget
/targetexact XXX
[/code]

So it should be immune to Mongoose versus Mongoose Pup, but that should be tested.

I am not aware of a way to use NPC ids to target, but I will research the topic.

I think we can alter the |U| tag to not just take an item, but a command. That way you could write |U|love| and it would /love the current target.

Oh, also I was thinking…Oh, also I was thinking… Since the Dancing Water Skimmer phases out (I presume after you hit 90 and start doing Golden Lotus dailys), could it be added into one of the leveling guides with a rank of 3? At that rank, it could be skipped as an unneccessary quest step.And this suggestion further promotes the idea of putting some achieves into the guides at |RANK|3| for the completionists out there. Achieves such as this one, explorer achieves, etc. Any thoughts on this?

A few issuesI ran through this guide once to test it out, and there are a few issues. The TomTom arrow doesn’t update after you /love your critter. The guide updates, just not the arrow. The |T| tag only uses a name to target? There may be issues with that with critters such as the Mongoose, because there is also a Mongoose Pup in the area as well. Could it use an npc id (from Wowhead) to target creatures? Also, is there a tag or another way to click a button to emote /love? That would be handy.Feel free to edit this, as it is a first draft. Better |N| tags? Better route to take?