search

Caraxi / MacroChain

10 stars 17 forks source link

Error: 'No macro is running' when trying to use /nextmacro at the end of a macro. #4

Open Lgutsfeld opened 3 years ago

Lgutsfeld commented 3 years ago

/nextmacro won't work reliably to transition between crafting macros after the last update. Sometimes it works, sometimes it doesn't. Not sure why you changed it in the first place, as there wasn't really an issue with the old implementation in the first place from what I can tell.

phildowney commented 3 years ago

Also having this issue.

Sharparam commented 3 years ago

I am also interested in why the original /runmacro was limited. Did SE change something in the game that made it not work anymore?

ghost commented 3 years ago

/nextmacro won't work reliably to transition between crafting macros after the last update. Sometimes it works, sometimes it doesn't. Not sure why you changed it in the first place, as there wasn't really an issue with the old implementation in the first place from what I can tell.

Nexmacro's issue is because some state information doesn't get set when Dalamud boots up initially, so until you've executed one macro, the plugin system doesn't know about whether there's a macro running or not, and it appears to default to "no".

The original reason given in the commit message for the change to runmacro was because of "Balance" in the author's own words.

Also, no commits in 2 months, pull request that's been open for just as long, Abandonware much?