Skip to content
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

Füge Suche nach Bekanntmachungen hinzu. #14

Merged
merged 6 commits into from
Aug 13, 2021

Conversation

PJUllrich
Copy link
Contributor

Anbei die Funktionalität, um die Bekanntmachungen des Handelsregister zu durchsuchen.

Bezieht sich auf diese URL: https://www.handelsregisterbekanntmachungen.de/?aktion=suche

Ich habe die Logik bzgl. der Suche im Handelsregister in eine Registrations-Klasse verschoben, um die Handelsregister-Klasse minimal zu halten.

Ich bitte einmal um Feedback.

@LilithWittmann
Copy link
Member

LilithWittmann commented Aug 13, 2021

Nice! Is there an option to automatically get the publications on a company as via the VÖ Link in the Handelsregister?

So that we maybe could have something like that

from deutschland import Handelsregister
hr = Handelsregister()
# search by keywords, see documentation for all available params
result = hr.search(keywords="Deutsche Bahn Aktiengesellschaft")
print(result[1].name)
print(result[1].publications)

As the idea of these abstractions is really that people shouldn't need to understand the website the data is scraped from.

@PJUllrich
Copy link
Contributor Author

Die Details einer Veröffentlichung zu holen steht bei mir als nächstes auf der Liste. Ich wollte nur schonmal dieses hier zur Review anbieten.

Ich mag die Idee einer Option "Hole Firmeninformationen mit Bekanntmachungen" und werde diese nach dem Holen der Details angehen 👍

@LilithWittmann LilithWittmann merged commit fed57b1 into bundesAPI:main Aug 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants