Describe the bug
This addon does not work for milling in WOTLK classic.

How to reproduce
Get stacks of herbs, press key combination and select herbs. Herbs just flash all of a similar type, no action is taken.

What did you expect?
The stack of herbs to be milled

I verified disenchant is still working on a different toon, so it appears to be an issue with milling specifically, probably because this build is for TBC. Prepatch also launches tomorrow so you should update the TOC if you have a chance.

Added full support for the wotlk herbs, ores, lockboxes etc in LibProcessable, please try it out and see if you have any issues and that the skill level check works correctly.

A packaged version can be found under the "Artifacts" header here:
https://github.com/p3lim-wow/LibProcessable/actions/runs/3002992432

@adnedelcu I know it's a bit early, but any chance you could check in-game if Fire Seed is millable? The item existed until 3.0.8 when WotLK was current, but wowhead lists the item as available in the game at classic's 3.4 patch level.

If it exists and is millable, I'd very much like to know what skill level is required to mill it.

Haven't gotten to it yet, but will keep an eye out for it.

atm the issue I get is this lua error with GetProfessions().

5x ...dOns\Molinari\libs\LibProcessable\LibProcessable-55.lua:383: attempt to call global 'GetProfessions' (a nil value)
[string "@molinari\libs\LibProcessable\LibProcessable-55.lua"]:383: in function <...dOns\Molinari\libs\LibProcessable\LibProcessable.lua:380>

@keyboardturner which game version?

sorry wrath classic, if the above version was not intended for wrath then my mistake

Someone would have to submit patches for the herbs, ores and lockboxes for this in https://github.com/p3lim-wow/LibProcessable, in a compatible manner, as I don't have any intention to play classic.

I'm just making sure, as there are 4 current game versions available :P

Created a patch to add the skill in the profession list for classic. I have tested only on a small version of the plants since my skill is not high enough.

p3lim-wow/LibProcessable#50

Thanks @adnedelcu!

I assume from that PR that milling works fine in Molinari, although herb compatibility testing was limited?

@danjohnso could you also test this by replacing libs/LibProcessable with the one from the link below, under the "Artifacts" section?
https://github.com/p3lim-wow/LibProcessable/actions/runs/2998725362

Yes, exactly. I only tested up to Mageroyal. I have seen that there is a piece of code for checking if the player has the profession high enough for it, but for all plants is set to 1.
That could be improved in the future. (I can try to do it tonight, but I need to find where I can get that information from)

In retail all herbs and most ores can be milled/prospected with 1 skill in the profession.
I'll go through the history and pull the old numbers out and use them.

I'd also have to re-implement the skill logic, which I've failed to do before (see p3lim-wow/LibProcessable#14).

That should do it, feel free to test and report back.

Hello, i just tested it, i added : p3lim-wow/LibProcessable@0755eda

And no errors, and it works ! ty ! :)

Updated my LibProcessable, will check today when servers come back up