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

Cardboard Backpack #321

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

Cardboard Backpack #321

wants to merge 3 commits into from

Conversation

VascoUP
Copy link

@VascoUP VascoUP commented Dec 18, 2016

Hello,

We added a new backpack, based on your cardboard items. It's a cardboard backpack!

Unfortunatly, there are still some things missing. The backpack doesn't have an icon when it's in your inventory, and the name ends with a '.name'. We couldn't find a way to fix both "errors" (if you can call them that).

The backpack has a recipe:
" o "
"ooo"
"ooo"
, 'o' being a cardboard sheet.

If you find this idea interesting we'd be glad to fix everything.

Thank you,
Vasco Pereira

@VascoUP VascoUP changed the title Submission Cardboard Backpack Dec 18, 2016
@copygirl
Copy link
Owner

This really is an interesting idea and goes well with the theme of having lots of cardboard related items. Though, I don't know about its purpose necessarily. I assume a cardboard backpack would have less slots than a regular backpack..? Considering slimeballs are often more difficult to find than leather and gold, I feel like it should have some sort of mechanic or other upside to make them worth considering.

I was looking through the commits and it appears the actual item, block and tile entity are missing.

As I already hinted at in a response to your other PR (#319), I would rather not do anything BetterStorage related, unless it is reimplementing features as a new mod. Continuing to maintain this mod, verify PRs and release updates (not to mention I'd feel like I'd have to fix some of the bugs and problems), is more than I can handle. I do however encourage you to continue your fork of the project, and if you'd like, I can link to it in this repo's readme and the forum thread.

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

Successfully merging this pull request may close these issues.

2 participants