Skip to content
This repository has been archived by the owner on Aug 27, 2018. It is now read-only.

Monitors addition #71

Merged
merged 7 commits into from
Nov 19, 2016
Merged

Monitors addition #71

merged 7 commits into from
Nov 19, 2016

Conversation

UnseenFaith
Copy link
Owner

The Hall Monitors have arrived. 😄
Adds:

  • loadCommandMonitors (loads monitors)
  • runCommandMonitors (runs monitors if they're enabled)
  • profanity.js (First Monitor, disabled by default to prevent issues)

Modifies:

  • App.js (Now Runs monitors before the checking of the command && ignores its own messages to prevent an infinite loop... might be useful if we prevent all bots messages.. //givemesuggestions)

@UnseenFaith
Copy link
Owner Author

@CyberiumShadow Not an error/issue or anything but just wanted to say I left
console.log(reason.stack); in there by mistake. This is the thing you said we might want to consider adding into core komada for errors. Your call if you want to leave it in or not.

@CyberiumShadow
Copy link
Collaborator

@UnseenFaith Ignoring bot messages can be done by checking if message author is a bot
if (msg.author.bot) {return;}

@CyberiumShadow
Copy link
Collaborator

Implement "Monitors"

@CyberiumShadow CyberiumShadow merged commit b396018 into UnseenFaith:indev Nov 19, 2016
@UnseenFaith UnseenFaith deleted the monitors-addition branch November 21, 2016 00:15
CyberiumShadow added a commit that referenced this pull request Nov 21, 2016
* Added padding to help command

* Removed one line function

* Added padding to help command

* Custom Permission Roles Support

* Adds member usage type.

* parseUsage fix and member usage type

* Allows multi word values to be used

* Fix random missing periods, spelling, etc.

* bot owner commands only...

* Fix for Core 'conf' command

* Adds Boolean Usage type and reverts old boolean fix

* Reverted again and made value checking better.
Will now correctly change depending on the type of the key.
Set can now set a boolean key as 'false'

* Quick fix for disable.js not disabling commands in
disabledCommands array

* Fix for default config being 'out of sync' on updates

* Better 'Fix', which allow you to use setKey, and not
have your default options overwritten, but still add
missing keys if you don't have them

* Updated forEach per Ao's recommendation

* Download.js VM "Cannot Find Module" Fix

* Readded Cybers selfbot option && fix for it

* If command is now selfbot only and bot is not a selfbot,
The bot will now uninstall the modules it previously installed.

* Selfbot commands now correctly load or not if selfbot

* Bumping Komada Core to use D.JS Indev

* Eslint/Beautify everything~

* Updated ESLint to allow Template Literals

* Fix Resolves/Rejects to stop executing further code

* Delete .jsbeautifyrc

* Update .eslintrc.json

* Update package.json

* Update package.json

* Various Changes.

* Update .eslintrc.json

* Fixed

* Conf.js Fix && Eslint

* The Great Conf Fix

* Update confs.js

* Eslinting Patch

* ESlint them all

* Fucked up?

* Semver Patch

* Fixes

* Bumped

* Monitors addition (#71)

* Monitors Loading && Monitors Running

* Message Event ignores bot messages and runs monitors

* Profanity- 1st Monitor Example

* Ignores Bots, Outputs Error Stacks into console

* Semver minor bump

Monitors

* removal of max-len from linter

* oops. I mean semver patch

* Better Support for Arrays/Ints/Booleans/Strings

* Update package.json

* Monitors/Inhibitors Uncaught Exception fix

* Verison Bumps

* Fixes Galore

* Aye Fixed Komada not listening to itself

* Reload all the things! (#75)

* Fixes last eslint issues and codeblock undefined issue

* New Reload.js Command.

* Reload Function! Reload all the pieces now!

* Fixes for Function/Command reload.js

* Semver Minor. Preparing for NPM release

* Create CHANGELOG.MD

* Rename CHANGELOG.MD to CHANGELOG.md

* Changelog Released.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants