Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable migration of long inserters #74

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Zevell
Copy link

@Zevell Zevell commented Nov 23, 2023

What

Removed the migration of long inserters into fast inserters found in this migration JSON file.

Why

Because the migration affects all blueprints and changes all items within the blueprints into their replacements, it messes with blueprints too much because there are no upgraded versions of the long inserters specifically, with the default longer pickup and insert positions.

If this migration is left in, it will "break" a lot of blueprints, as there is no swap-in replacement for the long inserter, due to the reasons mentioned above.

Break?

Blueprints will no longer function as expected, as the pickup and insert positions of the mk3 inserter (the long inserter's replacement) are different from the long inserter.

Possible future solutions

  • Add a startup setting to enable the replacement of the long inserters for those who want it.
  • Create upgraded tiers specifically for the long inserters.

## What
Removed the migration of long inserters into fast inserters found in this migration JSON file.

## Why
The reason being that it messes with blueprints too much, and there are no upgraded versions of the long inserters specifically, with the default longer pickup and insert positions.

If this migration is left in, it will "break" a lot of blueprints, as there is no swap-in replacement for the long inserter, due to the reasons mentioned above.

### Break?
What I mean by "break" is; blueprints will no longer function as expected, as the pickup and insert positions of the mk3 inserter (the long inserter's replacement) is different to the long inserter.

## Future solutions
- Add a startup setting to enable the replacing of the long inserters for those that want it.
- Create upgraded tiers of specifically the long inserters.
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.

1 participant