Skip to content
This repository has been archived by the owner on Sep 29, 2022. It is now read-only.

Handle "message" variable being undefined/null #87

Open
acollierr17 opened this issue Mar 19, 2022 · 0 comments
Open

Handle "message" variable being undefined/null #87

acollierr17 opened this issue Mar 19, 2022 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@acollierr17
Copy link
Member

https://github.com/suggestionsbot/suggestions-bot/blob/9917b6750166ef665009da335a5f84e2d15621af/src/commands/Suggestions/suggest.js#L114

This is somewhat of regression of #64. What I should do in this case is in the try block, I should have a conditional checking for if message still exists before doing any actions on it. Otherwise, I should ignore it altogether. This would also be accounted for in the catch block.

@acollierr17 acollierr17 added the bug Something isn't working label Mar 19, 2022
@acollierr17 acollierr17 self-assigned this Mar 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant