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

Allow IgnoreCheckPredicate to return a promise #155

Open
dj346 opened this issue Sep 1, 2020 · 0 comments · May be fixed by #157
Open

Allow IgnoreCheckPredicate to return a promise #155

dj346 opened this issue Sep 1, 2020 · 0 comments · May be fixed by #157

Comments

@dj346
Copy link

dj346 commented Sep 1, 2020

It would be nice if IgnoreCheckPredicate in ignorePermissions in CommandHandler allowed for promises

Just thought it would be nice; I was able to integrate my database nicely for the prefix doing

prefix: async (msg: Message) => {

            var serverData = await client.db.getRepository(ServerSettings).findOne({
                guild: msg.guild.id
            });

            return serverData.prefix != null ? serverData.prefix : prefix
        }

But wasn't able to do the same for ignore permissions because it doesn't allow for promises.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants