Skip to content

Add a step to return to GC barrack before starting Artisan crafting.#34

Open
nirjiang wants to merge 6 commits into
Ashylila:masterfrom
nirjiang:master
Open

Add a step to return to GC barrack before starting Artisan crafting.#34
nirjiang wants to merge 6 commits into
Ashylila:masterfrom
nirjiang:master

Conversation

@nirjiang

@nirjiang nirjiang commented Jul 8, 2026

Copy link
Copy Markdown

Add an option in settings: Return to GC Barrack prior to Artisan crafting, and then execute the crafting process.|
Avoid prolonged, repetitive actions before GC to prevent being reported.

螢幕擷取畫面 2026-07-08 220745

@Ashylila

Ashylila commented Jul 8, 2026

Copy link
Copy Markdown
Owner

Thanks for the submission! It looks like the rebase for this PR dropped the first commit from #33 (9a6333d, "Add barracks return before autogather follow-up"), so five files are missing — most importantly TheCollector/Utility/GrandCompanyBarracksReturnHandler.cs (the whole 270-line handler), plus the small changes to FrameRunnerPipelineBase.cs, ServiceWrapper.cs, PluginState.cs, and PluginStatePresentation.cs.

As it stands the branch won't compile: AutomationHandler.cs still injects GrandCompanyBarracksReturnHandler and calls _barracksReturn.Start(), but the class isn't in the diff.

@nirjiang

nirjiang commented Jul 8, 2026

Copy link
Copy Markdown
Author

It should be ready to merge and compile now. I completely messed up the repo just now.

@Ashylila

Copy link
Copy Markdown
Owner

Sorry for the late response but, been dealing with something person but I'll try to merge it this week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants