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

added documentation for OFF #891

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Svetlana-T
Copy link
Contributor

Credit to KindOne at #libera for mentioning this is not documented

Credit to KindOne at #libera for mentioning this is not documented
@RealKindOne
Copy link

After reviewing the chanserv/set_mlock source code and a bit of testing I have determined that I am wrong on that command.

You only need cs set #channel mlock remove the mlock. Anything invalid like OFF, nonexistent modes, oper-only modes (while not oper), has the same effect and remove the mlock.

There are a few other command's that accept multiple ways to disable them, but they are not documented on how:

ChanServ SET EMAIL
ChanServ SET ENTRYMSG 
ChanServ SET URL

They accept , NONE, and OFF

if (!parv[1] || !strcasecmp("OFF", parv[1]) || !strcasecmp("NONE", parv[1]))

if (!mail || !strcasecmp(mail, "NONE") || !strcasecmp(mail, "OFF"))

if (!url || !strcasecmp("OFF", url) || !strcasecmp("NONE", url))

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.

None yet

2 participants