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

Adding support for Lipizzanerheimat App, Austria #3209

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

wehrmannit
Copy link

  • Add support for Lipizzanerheimat, Austria

  • Format documentation

  • reformatting + ./update_docu_links.py

Uses guest credentials that were provided in the code.

Town Website
Bärnbach baernbach.gv.at
Edelschrott edelschrott.gv.at
Geistthal-Södingberg geistthal-soedingberg.at
Hirschegg-Pack hirschegg-pack.gv.at
Kainach kainach.at
Köflach koeflach.at
Krottendorf-Gaisfeld krottendorf-gaisfeld.gv.at
Ligist ligist.gv.at
Maria Lankowitz maria-lankowitz.at
Mooskirchen mooskirchen.at
Rosental a.d Kainach rosental-kainach.at
Sankt Martin am Wöllmißberg st-martin-woellmissberg.gv.at
Söding-Sankt Johann soeding-st-johann.gv.at
Stallhofen stallhofen.gv.at
Voitsberg voitsberg.gv.at

Comment on lines 307 to 309
raise SourceArgumentNotFound(
[], "No data found for the provided arguments.")

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same as above

@wehrmannit wehrmannit requested a review from 5ila5 December 19, 2024 19:00
Comment on lines +268 to +270
if (community_id == None):
raise SourceArgumentNotFound(
[], "Town/Street combination not found.")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This exception does not work like this. (see above)

You could raise the SourceArgumentNotFoundWithSuggestions exception in the fetch_communities function to provide feedback which towns would be valid

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.

2 participants