-
-
Notifications
You must be signed in to change notification settings - Fork 18
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
Add Exception Handling #8
Comments
@VickiLanger we can send a report email in case of exception, and keep on restarting the server after a configurable amount of time. |
@sroy8091 Yes, an email would work. That makes sense. I don't see why we couldn't restart the server until it works. |
@VickiLanger can we put the specific |
@achoudh5 that seems like it would make sense. Honestly, I have not done any exception handling outside of some practice. So, I'm not sure what it normally looks like. |
If I may ask, what I understood by |
Yes. |
Awesome! so, can we use an edge case file like if a user sends an |
Yes. That makes sense. An empty tweet definitely shouldn't happen. |
Is that the only case you want to consider for now? I can create a PR for that if it's fine |
That's the only thing I can think of. Go for it |
Need to add exception handling to tweet.py
Don't forget to follow the bot on Twitter to see the hard work you put in.
The text was updated successfully, but these errors were encountered: