Skip to content
This repository has been archived by the owner on Sep 17, 2022. It is now read-only.

Fix autolathe runtime when using custom materials #110

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

KubeRoot
Copy link
Contributor

@KubeRoot KubeRoot commented Aug 2, 2021

About The Pull Request

The autolathe tries to award an award to the person using it when they print something using custom materials other than glass and iron. With the bee tgui autolathe, the user printing was no longer passed to the make_item proc.

Because of this, it seems to break down completely after printing such items.

This PR tracks all users that have requested an item to be printed and tries to award the achievement to every user that requested the specific design.

The PR is completely untested, besides running dreamchecker, because it's too damn late for me to be testing this right now. Don't merge without testing yourself, otherwise wait for me to get around to it, which is annoying because it involves switching BYOND versions.

The PR has been tested, after a fix should be good to merge.

Why It's Good For The Game

Bugfix good.

Changelog

🆑
fix: Autolathe no longer breaks when printing items with unusual custom materials
/:cl:

@Laboredih123
Copy link

ill try and test it

@Laboredih123
Copy link

realized i dont know how to make custom material tools

@KubeRoot
Copy link
Contributor Author

KubeRoot commented Aug 2, 2021

realized i dont know how to make custom material tools

I believe toolboxes use custom materials, you can use those

@Laboredih123
Copy link

ill t r y

@Laboredih123
Copy link

vscode being shitty and using 514 aghhhhhhhhhhhhh

@Laboredih123
Copy link

is it just me or is the server no start

@Laboredih123
Copy link

atoms isnt loading

@Laboredih123
Copy link

prob just me

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants