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

fix drinking hat being able to store everything #77243

Merged
merged 1 commit into from
Oct 22, 2024

Conversation

NetSysFire
Copy link
Member

Summary

None

Purpose of change

<horis> Crafted a drinking hat to keep 500ml of projectiles on my head. It's like a rock and a chunk of steel lol

Fun use, made me chuckle, but definitely not intended.

Describe the solution

Thank you Renech for providing a quick solution.

Adds "max_item_length": "1 mm" to only permit liquids. Or finely folded napkins if you really want to.

Describe alternatives you've considered

Instead have a high turn cost for unloading items, but this applies to liquids, too, so this is not really viable.

Testing

Applied change locally. Can not insert rocks anymore but liquids still go in fine. And napkins. And apparently also boxer shorts and socks. Now the only thing missing is complimentary old sock juice.

Additional context

This player was sad and wanted an alternative to use. So if anyone knows clothing that allows you to store things on your head, that might be a good thing to contribute via a PR.

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Items: Armor / Clothing Armor and clothing json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Oct 22, 2024
@AudBobb
Copy link
Contributor

AudBobb commented Oct 22, 2024

image
image
image
image

I can look at adding one of these in tomorrow if we seriously want to add in a hat with a pocket lol. Would probably go with the fanny pack one, although the womens coin purse cap is mighty attractive xD

@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Oct 22, 2024
@Maleclypse Maleclypse merged commit defd017 into CleverRaven:master Oct 22, 2024
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions Items: Armor / Clothing Armor and clothing [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants