ickputzdirwech / Shortcuts-ick

Adds a host of useful shortcuts to improve QOL.
https://mods.factorio.com/mod/Shortcuts-ick
MIT License
6 stars 3 forks source link

Fix crashing for latest version of Advanced Artillery Remotes Continued (4.0.x) #13

Closed azaghal closed 1 year ago

azaghal commented 1 year ago

The purpose of this pull request is to introduce fixes/updates for the latest version of Advanced Artillery Remotes Continued (4.0.x).

The changes are required because the Advanced Artillery Remotes Continued has introduced a more robust support for multiple cluster remotes, and it has changed the phase in which the cluster remote prototypes are created.

Possible caveats in the pull request:

azaghal commented 1 year ago

Given the caveats, I'm available to make further changes to the PR as you see appropriate, although some things might not be the smoothest.

I was maybe thinking if it would make sense for the Shortcuts mod to support some form of interface that 3rd-party mods could invoke for registering shortcuts etc, but that'd be a bit more work down the line. A positive thing for 3rd-party mods that integrate in this manner would be ability to hide some of their internals from the Shortcuts mod (therefore reducing breakage), and they could also cope with dynamic processing.