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

Hole Details einer Bekanntmachung/Veröffentlichung #16

Merged
merged 2 commits into from
Aug 15, 2021

Conversation

PJUllrich
Copy link
Contributor

Anbei noch ein kleiner PR um die Details einer Bekanntmachung zu holen. Ich bitte um Feedback.

Als nächstes steht jetzt die hole Firma mit Bekanntmachungen/Veröffentlichungen auf der Liste.

@LilithWittmann
Copy link
Member

Das sieht grundsätzlich ganz gut aus, scheint aber gerade nur mit python 3.9(?) zu laufen.

@PJUllrich
Copy link
Contributor Author

PJUllrich commented Aug 15, 2021

Ah, sorry da geht doch was kaputt auf meiner Seite. Werde das heute Abend fixen.

@jugmac00
Copy link
Contributor

@PJUllrich du benutzt removeprefix das erst mit Python 3.9 eingeführt wurde (PEP 616).

AttributeError: 'str' object has no attribute 'removeprefix'

@PJUllrich
Copy link
Contributor Author

@PJUllrich du benutzt removeprefix das erst mit Python 3.9 eingeführt wurde (PEP 616).


AttributeError: 'str' object has no attribute 'removeprefix'

Ah ja stimmt, danke dir. Das hatte ich später schon ersetzt durch replace, aber habe es hier wohl vergessen. Werde es später anpassen.

@PJUllrich
Copy link
Contributor Author

@LilithWittmann Fix ist drin und jetzt sind alle Tests ✅

@LilithWittmann
Copy link
Member

Niiice!

@LilithWittmann LilithWittmann merged commit 138eb35 into bundesAPI:main Aug 15, 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

3 participants