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

[ARDMediathekBridge] Switch to JSON-API #1

Closed
wants to merge 1 commit into from

Conversation

Mar-Koeh
Copy link

The html-based screen scraping method seems to have stopped working. I could not get a non-empty list of items out of the bridge at the current revision of the ARDMediathek web app.

Switching to the JSON-API restores functionality – although there is still room for improvement. The following has not yet been added:

  • Add show's title to feed title
  • Understand API better and mimic webapp more accurately
  • Possibly depaginate in order to show all (instead of the first few) items

The html-based screen scraping method seems to have stopped working. I could not get a non-empty list of items out of the bridge at the current revision of the ARDMediathek web app.

Switching to the JSON-API restores functionality – although there is still room for improvement.
@nabakolu
Copy link

Using this commit, I still only get empty item lists

@Mar-Koeh
Copy link
Author

Mar-Koeh commented Jan 3, 2022

Using this commit, I still only get empty item lists

With which input? Here are some I tried successfully

@nabakolu
Copy link

nabakolu commented Jan 3, 2022

Must have been human incompetence, as it magically works now.
+1 for the commit

@Mar-Koeh
Copy link
Author

Mar-Koeh commented Jan 3, 2022

Must have been human incompetence, as it magically works now.

When copying a URL to a specific season, one needs to extract the show ID itself.

+1 for the commit

Reviewers in upstream requested some more changes. See the commit there as well: RSS-Bridge#2380

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