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

Wooden pulley fixes #60285

Merged
merged 2 commits into from
Aug 19, 2022
Merged

Wooden pulley fixes #60285

merged 2 commits into from
Aug 19, 2022

Conversation

Sathra225
Copy link
Contributor

Summary

Bugfixes "Wooden palisade pulley actually works."

Purpose of change

Missed how the gates in-game actually worked (gates.json) so the wooden pulley didn't actually work. Fix that and also adds a looks_like field for it.

Describe the solution

Adds t_palisade_pulley_wood entry in gates.json and adds a looks_like field for the wooden version the same as the metal.

Describe alternatives you've considered

Purely decorative rope pulley systems?

Testing

Made changes, checked it had a sprite and opened gate.

Additional context

I...may have forgotten to test the wooden version in #59880.

looks_like to wooden pulley
wooden pulley to actually work
@github-actions github-actions bot added [JSON] Changes (can be) made in JSON <Bugfix> This is a fix for a bug (or closes open issue) astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Aug 18, 2022
@dseguin dseguin merged commit 3426b9d into CleverRaven:master Aug 19, 2022
@Sathra225 Sathra225 deleted the woodpulleyfixes branch August 19, 2022 22:54
pjf added a commit to pjf/Cataclysm-DDA that referenced this pull request Aug 21, 2022
* origin/master:
  Prepper NPC quest follow-up (CleverRaven#59515)
  Adds a new widget for th structured sidebar with all midsize bodygraphs (i.e. the ones structured uses for "Health + Status" et al) side by side with nothing else, for use with Health + Overmap because I like the way the midsize bodygraphs look and don't want to have to choose just one of them to display (CleverRaven#60244)
  [DinoMod] slower zombie dino upgrades (CleverRaven#60311)
  [Xedra Evolved] Adds itemgroups for spell items (CleverRaven#60296)
  Update mutations.json (CleverRaven#60295)
  Fix typos (CleverRaven#60269)
  Wooden pulley fixes (CleverRaven#60285)
  Update landscaping.json (CleverRaven#60299)
  Update martialarts.json (CleverRaven#60305)
  fix: 'person' monster silently vanishes after touching character (CleverRaven#60270)
  fix: prevent use of non valid creature pointer (CleverRaven#60272)
  Nested Recipes - Ropes, Bundles, Shelled Nuts (CleverRaven#60297)
  Nested Recipes - Chain Armor (CleverRaven#60268)
  Nerf isherwood dandelion quest rewards (CleverRaven#60273)
  Animal ports from TropiCata to Mainline + egg density fixes for insects and birds (CleverRaven#60234)
  Modular Defense Anchor is SOFT (CleverRaven#60279)
Hirmuolio pushed a commit to Hirmuolio/Cataclysm-DDA that referenced this pull request Aug 27, 2022
* Update terrain-mechanisms.json

looks_like to wooden pulley

* Update gates.json

wooden pulley to actually work
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 <Bugfix> This is a fix for a bug (or closes open issue) [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.

2 participants