Skip to content

Cardboard Backpack - #321

Open
VascoUP wants to merge 3 commits into
copygirl:masterfrom
VascoUP:submission
Open

Cardboard Backpack#321
VascoUP wants to merge 3 commits into
copygirl:masterfrom
VascoUP:submission

Conversation

@VascoUP

@VascoUP VascoUP commented Dec 18, 2016

Copy link
Copy Markdown

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
Copy Markdown
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