Skip to content
This repository has been archived by the owner on Jan 28, 2023. It is now read-only.

[BUG] use of undefined variable #155

Closed
genzyy opened this issue Jan 4, 2022 · 1 comment · Fixed by #156
Closed

[BUG] use of undefined variable #155

genzyy opened this issue Jan 4, 2022 · 1 comment · Fixed by #156

Comments

@genzyy
Copy link
Contributor

genzyy commented Jan 4, 2022

Describe the bug

In this file customMessage is being used which is actually an undefined variable in the codebase and because of this its failing atleast for me when I run the bot locally on my discord server.

To Reproduce

  1. Run the bot locally according to the instructions given in the root folder readme.
  2. Try out the bot on your server by sending messages with restricted words and for warnings.
  3. You will get an error in your terminal/command line that the app has crashed and customMessage is undefined.

Expected Behavior

Instead of customMessage, use warningMessage which is defined in the file.

Screenshot/ Video

bot

Desktop (please complete the following information)

Device - Lenovo Legion Y530
OS - Arch Linux
Browser - Firefox/Discord
Version - Linux 5.15.12-arch1-1

Additional context

No response

@github-actions
Copy link

github-actions bot commented Jan 4, 2022

Hello 👋 Thank you very much for raising an issue 🙌 The maintainers will get back to you soon for discussion over the issue! Thank you for your contributions. 🚀

If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our Discord Server - we help and encourage each other to contribute to open source little and often 🤓 Any questions, let us know!

genzyy added a commit to genzyy/classroom-monitor-bot that referenced this issue Jan 4, 2022
add whitespace to whitelist and fix use of undefined variable
@genzyy genzyy mentioned this issue Jan 4, 2022
6 tasks
siddhant-khisty added a commit that referenced this issue Jan 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant