Sluimerstand / ProfessionShoppingList

Track recipes, reagents, cooldowns, and more!
4 stars 4 forks source link

Feature Request: Add per character saved lists #66

Closed LuckyBlob closed 1 year ago

LuckyBlob commented 1 year ago

Hello,

It would be nice to have "per character" lists.

Also, it would be nice to have a "saved list" that is immutable of the actual working list. It could also be updated (once set as the working list) with what is in your bags.

LuckyBlob commented 1 year ago

I have taken the liberty to modify the code to make this work. It's wrapped with -- Update nlawson -- end nlawson

I also took the liberty to add toggleable debug to have a semi callstack.

ProfessionShoppingList.lua_lua.txt ProfessionShoppingList.lua_toc.txt

LuckyBlob commented 1 year ago

I updated massively the code to fix all the bugs. I also fixed your "RunNextFrame" solution which could hang with the suggested callback on load solution which seems to work without hangs.

ProfessionShoppingList_lua.txt

Sluimerstand commented 1 year ago

Oh, wonderful! I'd taken a look at it before, but I really need to set aside some time to go through it properly because I didn't understand some bits and I need to test with it so I can understand how it works exactly. Hopefully I can do this soon. Thanks again for the effort! <3

LuckyBlob commented 1 year ago

Yeah, I kind of "did it for me", but I tried to be vocal with my variables. Don't hesitate to ask me questions, I can answer exhaustively if you need.

Sluimerstand commented 1 year ago

Alright, I found a way, way simpler solution to this problem. Feature will be added with the next release. :)