Skip to content

Add sandbox gamemode effects #35

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

Merged
merged 3 commits into from
May 10, 2024

Conversation

ReasonableScripts
Copy link
Contributor

the last few effects from the bottom of this page https://wiki.facepunch.com/gmod/Effects

@@ -66,5 +66,16 @@ GmodScripts_EffectWhitelist = {
acf_heat_explosion = true,
simfphys_engine_smoke = true,
eff_jack_plasmaarc = true,

balloon_pop = true,
Copy link
Member

Choose a reason for hiding this comment

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

balloon_pop can be abused to cause severe lag

Copy link
Contributor Author

Choose a reason for hiding this comment

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

is there a way to remove it from the pr then?

Copy link
Member

Choose a reason for hiding this comment

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

edit your branch to remove it

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

did that fix it?

@wrefgtzweve
Copy link
Member

i think some of the other effects can be bad too but i don't recognize them directly by effect name, so i suggest someone tests this before its merged

@ReasonableScripts
Copy link
Contributor Author

i just need to know what to remove :)

Copy link
Member

@sarahsturgeon sarahsturgeon left a comment

Choose a reason for hiding this comment

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

Apologies for the late response on this, it's been a busy year for us!

I'll approve this, and if we find that any of these effects are being abused we'll remove them.

Thank you for the contribution 👍

Copy link
Member

@sarahsturgeon sarahsturgeon left a comment

Choose a reason for hiding this comment

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

One quick styling fix and then we're good to go. I'll circle back to this next time I see it and merge it if you're too busy :)

@sarahsturgeon sarahsturgeon merged commit 2080e4a into CFC-Servers:main May 10, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants