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

Shredders #499

Merged
merged 14 commits into from
Oct 29, 2023
Merged

Shredders #499

merged 14 commits into from
Oct 29, 2023

Conversation

IData-RedactedI
Copy link
Contributor

About the PR

Added peaceable but non functional shredders
Hopefully this is enough functionality that others can add to

Why / Balance

Its on "THE LIST"

Media

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

image

IData-RedactedI and others added 10 commits October 14, 2023 18:50
*Circuitvend Sprite

*Amended vending_machines.yml to show sprite
Added nanotransen and syndicate shredder sprites
-Off
-Idle
-Shredding money
-not shredding anything
Even better JSON
It will gladly take your money, but nothing else
Based off the lathe, the shredder is spawnable and interactable.
@IData-RedactedI
Copy link
Contributor Author

Hey Checkraze, this is unfortunately as far as I can go when it comes to the shredders. I think whatever systems you want to add can be grafted onto the framework here though.

@Cheackraze
Copy link
Member

Sure thing! I won't have a chance to review for a day or two, but thank you very much for your contribution!

@Cheackraze
Copy link
Member

excellent. The current functionality is whatever, since this is planned to be a part of a much bigger system in the near future.

@Cheackraze Cheackraze merged commit badb160 into new-frontiers-14:master Oct 29, 2023
9 of 11 checks passed
@IData-RedactedI IData-RedactedI deleted the Shredders branch November 17, 2023 16:18
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