Skip to content

Scp3144 severed hands fix#205

Merged
louis1706 merged 4 commits intoExMod-Team:scpsl14from
Undid-Iridium:Scp3144-SeveredHandsFix
Nov 27, 2024
Merged

Scp3144 severed hands fix#205
louis1706 merged 4 commits intoExMod-Team:scpsl14from
Undid-Iridium:Scp3144-SeveredHandsFix

Conversation

@Undid-Iridium
Copy link
Copy Markdown

Scp3144 severed hands fix - probably

#197

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentations

Submission checklist

  • I have checked the project can be compiled
  • I have tested my changes and it worked as expected

Patches (if there are any changes related to Harmony patches)

  • [] I have checked no IL patching errors in the console

Other

  • Still requires more testing

#197

Comment on lines +123 to +127
/// <summary>
/// Damage caused by severed eyes.
/// </summary>
SeveredEyes,

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replace the Scp1344 with this one

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Double check me

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry i meaned than Scp1344 damage is not more used so replace the name of damagetype with the new one you made

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So good?

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just removed line 261 no ?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants