Skip to content
This repository was archived by the owner on Jul 16, 2022. It is now read-only.

Modlog command always returning disabled even when enabled#89

Merged
vladfrangu merged 1 commit intoGAwesomeBot:developmentfrom
vladfrangu:development
Feb 25, 2017
Merged

Modlog command always returning disabled even when enabled#89
vladfrangu merged 1 commit intoGAwesomeBot:developmentfrom
vladfrangu:development

Conversation

@vladfrangu
Copy link
Copy Markdown
Member

credit to my friend, SamSoft

Setting up modlog is easy. {prefix}modlog enable #channel. You can check it using {prefix}modlog.

Theres an issue, it wasn't checking for isEnabled. it was checking for an inexistent log_channel variable.

Replacing that with isEnabled fixes the issue (also, it goes red if its off, and green if its on.)

Spacing is just Atom 😛

@vladfrangu vladfrangu added reason: Bug Issues that are caused by a deviancy in GAB's behavior target: Commands Issues that target Command files s: Awaiting Feedback Issues which are awaiting additional feedback from maintainers or the community. reason: V4.0 Source Issues that are caused by AwesomeBot source code labels Feb 24, 2017
@vladfrangu vladfrangu added this to the v4.1.0 milestone Feb 24, 2017
Copy link
Copy Markdown
Collaborator

@HilbertGilbertson HilbertGilbertson left a comment

Choose a reason for hiding this comment

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

Looks fine to me.

@vladfrangu vladfrangu removed the s: Awaiting Feedback Issues which are awaiting additional feedback from maintainers or the community. label Feb 24, 2017
@vladfrangu
Copy link
Copy Markdown
Member Author

Merge-ready

@vladfrangu vladfrangu merged commit a5bed11 into GAwesomeBot:development Feb 25, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

reason: Bug Issues that are caused by a deviancy in GAB's behavior reason: V4.0 Source Issues that are caused by AwesomeBot source code target: Commands Issues that target Command files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants