search

jellyfin / TMDbLib

C#.Net library for TheMovieDB
MIT License
344 stars 128 forks source link

Fix for #411 #416

Closed SirSparkles closed 1 year ago

Hona commented 1 year ago

@LordMike Any chance you can merge this in? I went to make this same PR :D

Hona commented 1 year ago

Actually incomplete:

            RestRequest req = _client.Create("trending/tv/{time_window}");
            req.AddUrlSegment("time_window", timeWindow.ToString().ToLower());

Needs TV too.

Might be better to use EnumValue though instead?

SirSparkles commented 1 year ago

@Hona - Good calls, have updated the PR

LordMike commented 1 year ago

Looks good. Once it's merged, you can pull the beta packages from github.

Thanks @SirSparkles and @Hona.

SirSparkles commented 1 year ago

Can we release this on NuGet please?