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

Fix recycler eating materials (Salvage mains rejoice) #32144

Merged
merged 1 commit into from
Sep 18, 2024

Conversation

beck-thompson
Copy link
Contributor

@beck-thompson beck-thompson commented Sep 14, 2024

About the PR

When you clicked on a recycler it would store the materials in a storage that was impossible to access. Not quite sure why the storage is even on the grinder but it is being used to do something important looking (not quite sure what its doing😆).

Media

2024-09-13.23-57-32.mp4

Breaking changes

Changelog

🆑 Beck Thompson

  • fix: Recycler no longer allows basic materials to be inserted into it.

@github-actions github-actions bot added the Changes: No C# Changes: Requires no C# knowledge to review or fix this item. label Sep 14, 2024
@thebadman4662
Copy link

Does it fix this or stacks bigger than 1 still get stored?

@beck-thompson
Copy link
Contributor Author

Does it fix #25516 or stacks bigger than 1 still get stored?

Sorry! I forgot to reply to this... I did test it and I couldn't quite get what your saying? I think it could but I'm not sure

@thebadman4662
Copy link

Does it fix #25516 or stacks bigger than 1 still get stored?

Sorry! I forgot to reply to this... I did test it and I couldn't quite get what your saying? I think it could but I'm not sure

Recycling 2 stacks of 30 and 30 rods is same as 1 and 1 rods, just that 30s will take waaaay longer to get your 1 steel. Wondered if it could be connected to some kind of inner storage as I never knew there was one.

@slarticodefast
Copy link
Member

Thanks for the fix!

@slarticodefast slarticodefast merged commit 58119bc into space-wizards:master Sep 18, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: No C# Changes: Requires no C# knowledge to review or fix this item.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants