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

MP gaming #4698

Closed
wants to merge 1 commit into from
Closed

MP gaming #4698

wants to merge 1 commit into from

Conversation

Diegoflores31
Copy link
Contributor

@Diegoflores31 Diegoflores31 commented Oct 16, 2023

About the pull request

Disablers will now slow xenomorphs on hit and make a Jittery effect.
Impacting a human 3 times with a disabler will make it jittery for a while.

Explain why it's good for the game

Aliens usually are able to receive damage from electric shock i wanted to give it a snowflake effect.

humans will now shake when hit by the taser too.

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
add: disablers can now affect xenomorphs.
qol: marines now shake while tased with a disabler.
/:cl:

@github-actions github-actions bot added Feature Feature coder badge Quality of Life Make the game harder to play labels Oct 16, 2023
@realforest2001 realforest2001 added Balance You need to be a professional veteran game maintainer to comprehend what is being done here. Do Not Merge If you merge this PR, I will annihilate you labels Oct 16, 2023
@realforest2001
Copy link
Member

Holding for discussion.

@fira fira removed the Quality of Life Make the game harder to play label Oct 16, 2023
if(ishuman(M))
var/mob/living/carbon/human/H = M
H.disable_special_items() // Disables scout cloak
/datum/ammo/energy/taser/on_hit_mob(mob/morbius, obj/projectile/P)
Copy link
Contributor

Choose a reason for hiding this comment

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

This new var name is not better. Change both of them

@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Oct 18, 2023
@cm13-github
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@BeagleGaming1 BeagleGaming1 marked this pull request as draft October 19, 2023 00:25
@Zonespace27
Copy link
Contributor

At this time, I don't want MP weapons to be useful in HvX scenarios, so this PR will be closed. I am however alright with a low jitter effect when humans are tased.

@Diegoflores31 Diegoflores31 deleted the MPgaming branch October 30, 2023 05:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance You need to be a professional veteran game maintainer to comprehend what is being done here. Do Not Merge If you merge this PR, I will annihilate you Feature Feature coder badge Merge Conflict PR can't be merged because it touched too much code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants