Skip to content

Commit

Permalink
fix: multiple genre filtering now works (#3282)
Browse files Browse the repository at this point in the history
  • Loading branch information
OwsleyJr committed Jan 26, 2023
1 parent f4a22dc commit 5076938
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 5 deletions.
2 changes: 1 addition & 1 deletion overseerr-api.yml
Expand Up @@ -4418,7 +4418,7 @@ paths:
- in: query
name: genre
schema:
type: number
type: string
example: 18
- in: query
name: network
Expand Down
2 changes: 1 addition & 1 deletion server/api/themoviedb/index.ts
Expand Up @@ -85,7 +85,7 @@ interface DiscoverTvOptions {
voteAverageLte?: string;
includeEmptyReleaseDate?: boolean;
originalLanguage?: string;
genre?: number;
genre?: string;
network?: number;
keywords?: string;
sortBy?: SortOptions;
Expand Down
8 changes: 5 additions & 3 deletions server/routes/discover.ts
Expand Up @@ -356,7 +356,7 @@ discoverRoutes.get('/tv', async (req, res, next) => {
page: Number(query.page),
sortBy: query.sortBy as SortOptions,
language: req.locale ?? query.language,
genre: query.genre ? Number(query.genre) : undefined,
genre: query.genre,
network: query.network ? Number(query.network) : undefined,
firstAirDateLte: query.firstAirDateLte
? new Date(query.firstAirDateLte).toISOString().split('T')[0]
Expand Down Expand Up @@ -491,7 +491,7 @@ discoverRoutes.get<{ genreId: string }>(
const data = await tmdb.getDiscoverTv({
page: Number(req.query.page),
language: req.locale ?? (req.query.language as string),
genre: Number(req.params.genreId),
genre: req.params.genreId,
});

const media = await Media.getRelatedMedia(
Expand Down Expand Up @@ -770,7 +770,9 @@ discoverRoutes.get<{ language: string }, GenreSliderItem[]>(

await Promise.all(
genres.map(async (genre) => {
const genreData = await tmdb.getDiscoverTv({ genre: genre.id });
const genreData = await tmdb.getDiscoverTv({
genre: genre.id.toString(),
});

mappedGenres.push({
id: genre.id,
Expand Down

0 comments on commit 5076938

Please sign in to comment.