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

JsonUrl builder #1825

Open
wants to merge 1 commit into
base: nightly
Choose a base branch
from
Open

Conversation

hjone72
Copy link
Contributor

@hjone72 hjone72 commented Jan 14, 2024

Description

New builder: JsonUrl

I had a pretty specific use case here. I wanted to be able to use Tautulli's non api endpoint to generate a list of movies from the media_info endpoint. The non api version is a little more powerful and lets you set multiple sort orders. This JsonUrl module will let you process a list of rating keys from a URL.
The module is working but the checking it does isn't as advanced as the checking performed by the Tautulli module. A further enhancement would be to use the same method that Tautulli does to check each rating key.

Issues Fixed or Closed

  • Fixes #(issue)

Type of Change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

Checklist

  • My code was submitted to the nightly branch of the repository.

I had a pretty specific use case here. I wanted to be able to use Tautulli's non api endpoint to generate a list of movies from the media_info endpoint. The non api version is a little more powerful and lets you set multiple sort orders.

This JsonUrl module will let you process a list of rating keys from a URL.
@meisnate12
Copy link
Member

could you explain how to get to this jsonurl?

@hjone72
Copy link
Contributor Author

hjone72 commented Jan 29, 2024

The idea is more around flexibility. I'm using a custom query against Tautulli to generate a json file which I'm serving via a web server. As long as the JSON contains the rating key it should work.

@chazlarson
Copy link
Contributor

This should probably have some documentation added.

@YozoraXCII
Copy link
Contributor

@hjone72 do you plan on addressing the documentation suggestion? if not, can we close this?

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.

4 participants