-
Notifications
You must be signed in to change notification settings - Fork 51
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
Fix 'PlantUMLHTTPError' object has no attribute 'message' #9
base: master
Are you sure you want to change the base?
Conversation
Also, pass 'message' to super, so that it's displayed. (I suspect this has to do with Python 3 changes?)
`super` should get the current class as parameter
@dougn good thing you didn't notice this PR yet 😬 |
@jean It seems that the new version also has a potential issue. The |
I'm facing this errror via mkdocs if there are some syntax issues in the plantuml block. Is the PR somehow blocked? |
+1 |
I'm trying to use PlantUML in colab.research.google.com and can't get error handling to work properly because of this. Hoping you can merge this. |
At times it appears if the input UML Code has syntax errors. Please double-check the input UML Code. @jean @wonderchang @fuhrmanator |
@Shareef509 if you could see the rendered image (that's returned despite the syntax error) you could see exactly what the error in the plantuml syntax and fix it. "Double checking" is hard if you can't get the error information. |
I had the same experience, that's why shared if it helps anyone else. Thank you. |
Also, pass 'message' to super, so that it's displayed.
(I suspect this has to do with Python 3 changes?)