search

daemitus / SomethingNeedDoing

92 stars 67 forks source link

Select String needs a new entry #114

Closed ladyredxiv closed 1 year ago

ladyredxiv commented 1 year ago

With retainer menu changes, we need a 13th select string command added into the clicks.

MaximusMaraad commented 1 year ago

With retainer menu changes, we need a 13th select string command added into the clicks.

I was going to open this topic too kkkkkk

BeetleLord commented 1 year ago

Indeed, we can't select the option to exit retainer menus currently.

WilliamW1979 commented 1 year ago

I already saw this problem, fixed it. I tested it and I am TRYING to send him the update right now.

WilliamW1979 commented 1 year ago

Alright, I did a pull request to fix this. It does work because I am using it in developer mode thanks to not being able to update it for some reason on Dalamud's end. I just happened to have the fix on my end because I was working in developer mode because Dalamud is not updating a lot of plugins so I manually put some in like this one.

Just so you guys know, this wasn't an update for Something Needing Done. The update was for ClickLib to add more. You won't see the pull request in here because of that but don't worry, it is done and you should see the fix soon.

WilliamW1979 commented 1 year ago

Oh, I made it go up to 14 strings for now in the pull request ... I figure I add one just in case.

ladyredxiv commented 1 year ago

Thank yoooou!!