-
-
Notifications
You must be signed in to change notification settings - Fork 126
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Inventory Management issues (white items) #558
Comments
Gooday, Thanks for letting me know about the issue. I'm going to look into it, and see what's going on. |
Thanks! |
Firstly, after doing a quick manual test,
/run DataToColor:sell({"Refreshing Spring Water", "Crisp Spider Meat"}); works fine for me, the item is getting sold properly. So there must be other issue, with the timing. Proabaly executing the Secondly, the other macro doesn't work since blizzard had the wonderful idea to change old existing api, move functions from global namespace under specific ones, so in order to make it work, you have to be more verbose in the macro, which going to limit what you can actually achieve. So sadly that macro cannot fit in the 255 character limit. |
Ahhh, "DataToColor" is the name of the addon, I had configured a different name on initial setup so that wasn't working. All good now! Thank you for the quick help! |
Hey here's a working version of the item deletion macro, however it needs to include the namespace in order to call the Container related functions. So you have less items to include and still fit within the 255 characters In the readme page, the example also updated, thanks for the notice. /run c=C_Container for b=0,4 do for s=1,c.GetContainerNumSlots(b) do local n=c.GetContainerItemLink(b,s) if n and (strfind(n,"Slimy") or strfind(n,"Pelt") or strfind(n,"Mystery")) then c.PickupContainerItem(b,s) DeleteCursorItem() end end end |
Hey thanks for this. It seems like there is still a timing issue when trying to sell with the bot. It sells greys but doesn't seem to run the macro and exits out of the merchant pretty quickly. Any way I could configure that to give it more time ? |
Gooday, i've made some adjustments both addon side and the backend side.
After #562, the previously mentioned issue should be fixed, if you have time to checkout the change and give feedback, i would really appreciate your time. Let me know if you notice any problem regarding to this issue. Have a nice day! |
Hey! I'll give it a try when I get home today. Thanks again :) |
Just tried it out and it seems to be fixed. Well done! |
Description
Playing SoD
Using the suggested sell macro I'm unable to sell any white items. My grey items vendor, repair works, but I cannot sell any specified white items.
my macro:
Does not sell either of the described items. I have this bound to "C" with bindpad
I also cannot seem to delete these items with the suggested macro
Thoughts / suggestions ? Really sucks to not be able to clean up garb in my inventory...
Addon Version
1.7.50
World of Warcraft Client
Season of Mastery
Reproduction Steps
Last Good Version
No response
Screenshots
No response
Logs
[13:03:17:151 W] [AdhocNPCGoal ] No target found by cursor(Vendor, Repair, Innkeeper)!
[13:03:17:151 I] [AdhocNPCGoal ] Use KeyAction.Key macro to acquire target
[13:03:17:243 I] [AdhocNPCGoal ] Found Target!
[13:03:18:182 W] [AdhocNPCGoal ] Gossip no options! 877.0381ms
[13:03:18:182 I] [AdhocNPCGoal ] Merchant window opened after 877.0381ms
[13:03:23:182 I] [AdhocNPCGoal ] Merchant sell nothing! -5000.0933ms
[13:03:24:956 I] [NpcNameFinder] type = None | mode = Fuzzy
The text was updated successfully, but these errors were encountered: