-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
Conversation
@@ -66,5 +66,16 @@ GmodScripts_EffectWhitelist = { | |||
acf_heat_explosion = true, | |||
simfphys_engine_smoke = true, | |||
eff_jack_plasmaarc = true, | |||
|
|||
balloon_pop = true, |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if youre using the web editor you can do it here
https://github.com/ReasonableScripts/cfc_gmod_scripts/blob/patch-1/lua/cfc_gmod_scripts/effects_whitelist/sh_init.lua
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
did that fix it?
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 |
i just need to know what to remove :) |
the last few effects from the bottom of this page https://wiki.facepunch.com/gmod/Effects
There was a problem hiding this 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 👍
There was a problem hiding this 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 :)
the last few effects from the bottom of this page https://wiki.facepunch.com/gmod/Effects