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

[SchweinfurtBuergerinformationenBridge] Add new bridge #1610

Merged
merged 3 commits into from
Mar 24, 2022

Conversation

mibe
Copy link
Contributor

@mibe mibe commented Jun 14, 2020

This bridge retrieves the latest official news for citizens of the city Schweinfurt. Stuff like the results from the election, where the next traffic disturbances due to road construction will be, changed opening hours of the town hall, etc...

@mibe mibe marked this pull request as ready for review June 14, 2020 22:34
@somini
Copy link
Contributor

somini commented Jul 14, 2020

The URI should be a "regular" URL. If you need to generate a per-page URL, concatenate the base URL and the ?art_pager=%d parameterization.

I would set it as https://www.schweinfurt.de/rathaus-politik/pressestelle/buergerinformationen/index.html

LGTM otherwise.

@mibe
Copy link
Contributor Author

mibe commented Jul 18, 2020

Thanks for pointing that out. I didn't knew that beforehand.

mibe added a commit to mibe/rss-bridge that referenced this pull request Jul 18, 2020
@mibe
Copy link
Contributor Author

mibe commented Dec 30, 2020

So I was wondering if this PR is considered for merging? If not, is the geographical scope of this bridge just too local?

@somini
Copy link
Contributor

somini commented Dec 30, 2020

Not really, the maintainers are probably overworked, the issues and PR are piling up.

@Bockiii
Copy link
Contributor

Bockiii commented Mar 24, 2022

Just tested, looks good, everything works. Thanks!

@Bockiii Bockiii merged commit e89b428 into RSS-Bridge:master Mar 24, 2022
@mibe mibe deleted the SW-news branch April 2, 2022 21:40
Kwbmm pushed a commit to Kwbmm/rss-bridge that referenced this pull request Jun 17, 2022
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.

3 participants