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: "person" monster dissipates silently #60270

Merged
merged 1 commit into from
Aug 19, 2022

Conversation

dseguin
Copy link
Member

@dseguin dseguin commented Aug 18, 2022

Summary

None

Purpose of change

According to #59558 (comment) the new "person" monster is not intended to deafen the player when disappearing.

Describe the solution

Reverts the changes to boomer_claymore_suicide and moves them to a new spell effect.

Describe alternatives you've considered

Testing

  1. Spawn person
  2. Let person touch the player character
  3. person curses the player and disappears silently
(video)
2022-08-18.02-21-15.mp4

Additional context

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Mechanics: Enchantments / Spells Enchantments and spells labels Aug 18, 2022
@GuardianDll
Copy link
Member

GuardianDll commented Aug 18, 2022

I'd prefer to unify the suicide monsters effect (maybe even make it without this dirty trick with self killing spell), but thats also good

@github-actions github-actions bot added 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
@bombasticSlacks bombasticSlacks merged commit 326fae6 into CleverRaven:master Aug 19, 2022
@dseguin dseguin deleted the silent_person branch August 19, 2022 22:11
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
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 [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Mechanics: Enchantments / Spells Enchantments and spells
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants