-
Notifications
You must be signed in to change notification settings - Fork 67
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
Bundesanzeiger not working properly anymore #125
Comments
Nvm, Bundesanzeiger has downtime rn. But still, could a proper warning as an output be useful (if there are maintenance phases)? |
The bundesanzeiger API is not auto generated so it would be possible to add a simple check if the request to the server was successful before proceeding. Happy to merge the needed changes if you want to tackle it. |
Hey there, I'd like to tackle this issue if it's fine for you guys. Could you assign it to me? |
I merged your PR. Would be good to add an test case to see if the exection is thrown if the url is not availble :) E.g. test the __response method with an url which is not available. Something like this |
Sure, I'll create a new PR for this shortly 👍 |
Tests also merged. |
Running the following sample code:
throws the following error:
The text was updated successfully, but these errors were encountered: