b-morgan / Skillet

World of Warcraft addon
GNU General Public License v3.0
15 stars 6 forks source link

Alchemy Transmute #76

Closed Dr0xX3rZ closed 1 year ago

Dr0xX3rZ commented 1 year ago

Skillet 5.02

Hi,

In the transmute section 2 different problems:

It is not showing the number out of 7 for transmute available.

It is saying that Cooldown remaining is 48 DAYS

b-morgan commented 1 year ago

Can you provide a screenshot that shows the issue, please?

My alchemist is only level 47 and therefore doesn't have the Dragon Isles recipes. If you can demonstrate the issue with Shadowlands or lower recipes, then I'd have a better chance of reproducing it.

Also, Skillet-5.03-alpha5 (or later) has many changes. Not sure they will help in this case but it will be easier for me if we both are using the same code base.

Dr0xX3rZ commented 1 year ago

Hi,

Here are 2 screenshots showing in Skillet vs Wow UI. Meanwhile I will switch to alpha version.

https://imgur.com/a/MzkjhSX

b-morgan commented 1 year ago

The above imgur link is the same as the one in #74. You can paste images directly into this issue which will help document it.

Dr0xX3rZ commented 1 year ago

The above imgur link is the same as the one in #74. You can paste images directly into this issue which will help document it.

My bad. https://imgur.com/a/FY0rnPM

image image

b-morgan commented 1 year ago

I can't find any online documentation for C_TradeSkillUI.GetRecipeCooldown but a search of the BlizzardInterfaceCode reveals

local cooldown, isDayCooldown, charges, maxCharges = C_TradeSkillUI.GetRecipeCooldown(skill.id)

with a much more complicated function to display the cooldown text. I probably need to understand this function and duplicate it but I won't be able to test it so I'll need your help.

Dr0xX3rZ commented 1 year ago

sure, what can I do?

b-morgan commented 1 year ago

I'll write the new function and give you either individual files or a private release. You'll have to see if it works and provide debugging output if it doesn't. I'll include instructions on how to do that if necessary. It may take me a day or two to get this ready.

Alternatively, you can wait until I level my Alchemist to 58 so I can learn Dragon Isles recipes and then however long after that before I learn the transmute that is failing. You will likely to be waiting months 😀.

Dr0xX3rZ commented 1 year ago

ready to test when you're done coding!

b-morgan commented 1 year ago

Here is a replacement for your current copy of Skillet. The TOC says 5.03-alpha5+ and it is my current working version. It should do something sensible with cooldowns and contains the fixes from all of your other open issues. Please test as many different kinds of cooldowns as you can today and let me know which were successful and if any were unsuccessful. I'd like to publish a new alpha version tomorrow.

Skillet.zip

Dr0xX3rZ commented 1 year ago

Seems fine for now.

image image

b-morgan commented 1 year ago

Thank you for checking. The only ones I was able to test were ones that reset daily.

b-morgan commented 1 year ago

Skillet-5.03-alpha7 contains this fix.