-
-
Notifications
You must be signed in to change notification settings - Fork 6.4k
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
Error handling #1
Conversation
Hi @shijuleon, Could you add some styling to the HTTP requests errors? What I mean with styling is color, just like the other outputs. Other than that, everything seems great! |
Sure! I have added some quick changes. You can improve it if you wish. |
Could fix two things? 2. Could you do something with the error message? Its kinda messy. Instead of showing the error, could you show the name of the social media instead?
The error in the image is related to issue #9 |
Fixed the error printing. I don't think we need the option to enter username. As a command line parameter seems fine. What do you think? |
Could you remove the banner from the help message? In my opinion, it only looks good when the script is actually running not when just showing the help message. If you fix that, I will merge it. |
I have removed the banner. Just thought it looked fancy. |
Thank you! 😄 |
…umentation Update README.md
added browse functionality to notify.py
HTTP requests error handling
Also adding command line parameters