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

Shredder Sprites #442

Closed

Conversation

IData-RedactedI
Copy link
Contributor

About the PR

Added shredder sprirtes
-Both Nanotransen and Syndicate flavours

With animations for
-Off
-Idle
-Shredding Money
-Shredding Nothing

Media

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

shredder_scaled_6x_minified

*Circuitvend Sprite

*Amended vending_machines.yml to show sprite
@FoxxoTrystan
Copy link
Contributor

Shredder sprite... alone with nothing that uses it?
This just add extra files that wont be used...

I think sprites should be added with the full files as this is just "add unsued files until we add it"

Tho good work on those!

@IData-RedactedI
Copy link
Contributor Author

Rather than continue to fail at making this an object, I'm following Checkraze's suggestion and submitting the sprites so he can do his thing.

@Kesiath
Copy link
Contributor

Kesiath commented Oct 19, 2023

I hope this accepts paper too so I can shred people's hopes and dreams as SR. 👍

@Cheackraze
Copy link
Member

Shredder sprite... alone with nothing that uses it? This just add extra files that wont be used...

I think sprites should be added with the full files as this is just "add unsued files until we add it"

Tho good work on those!

this is a part of our sprite request issue #166, and therefore, i specifically requested the texture be added so that I can then code the functionality. So its just fine dw

@Cheackraze Cheackraze mentioned this pull request Oct 19, 2023
22 tasks
@Cheackraze
Copy link
Member

any issues with needing to update this? I could finish it if need be

Even better JSON
@IData-RedactedI
Copy link
Contributor Author

Aw man I buggered my master, gonna close this PR and set it up properly

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.

4 participants