-
Notifications
You must be signed in to change notification settings - Fork 74
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
feat: remove sex #395
base: dev
Are you sure you want to change the base?
feat: remove sex #395
Conversation
tested by french man, doesnt work |
It won't work cuz scp sl is started by local admin with own args. So we need to somehow put this args through LA (i don't remember if there is a way) or use other ways |
maybe with exiled loader config ? |
should be hidden, so exiled loader config doesnt work in this case |
I don't see an issue with this, as LocalAdmin does pass through arguments as long as they start with |
it doesnt, neither me nor yamato had it working |
My best guess as to why it doesn't work is that Exiled is loaded separately to the server application, so environment arguments would be null/empty. |
Why we even need to remove it? 💀 |
some hosting service wanted a way to remove it i showed than i wasn't against and Nameless did this PR i just said i didn't want an config for it but that would have been fine |
Description
Describe the changes
What is the current behavior? (You can also link to an open issue here)
What is the new behavior? (if this is a feature change)
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Other information:
Types of changes
Submission checklist
Patches (if there are any changes related to Harmony patches)
Other