Skip to content

Commit

Permalink
feat(frontend): recently added on discover
Browse files Browse the repository at this point in the history
also improves modal requests by not waiting for result from radarr/sonarr
  • Loading branch information
sct committed Nov 11, 2020
1 parent 7287a6a commit 06dc606
Show file tree
Hide file tree
Showing 8 changed files with 213 additions and 21 deletions.
54 changes: 54 additions & 0 deletions overseerr-api.yml
Original file line number Diff line number Diff line change
Expand Up @@ -736,6 +736,21 @@ components:
name:
type: string
example: 720p/1080p
PageInfo:
type: object
properties:
page:
type: number
example: 1
pages:
type: number
example: 10
pageSize:
type: number
example: 10
results:
type: number
example: 100

securitySchemes:
cookieAuth:
Expand Down Expand Up @@ -1847,6 +1862,45 @@ paths:
type: array
items:
$ref: '#/components/schemas/TvResult'
/media:
get:
summary: Return all media
description: Returns all media (can be filtered and limited) in JSON format
tags:
- media
parameters:
- in: query
name: take
schema:
type: number
nullable: true
example: 20
- in: query
name: skip
schema:
type: number
nullable: true
example: 0
- in: query
name: filter
schema:
type: string
nullable: true
enum: [all, available, partial, processing, pending]
responses:
'200':
description: Returned media
content:
application/json:
schema:
type: object
properties:
pageInfo:
$ref: '#/components/schemas/PageInfo'
results:
type: array
items:
$ref: '#/components/schemas/MediaInfo'

security:
- cookieAuth: []
6 changes: 4 additions & 2 deletions server/entity/MediaRequest.ts
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,8 @@ export class MediaRequest {
});
const movie = await tmdb.getMovie({ movieId: this.media.tmdbId });

await radarr.addMovie({
// Run this asynchronously so we don't wait for it on the UI side
radarr.addMovie({
profileId: radarrSettings.activeProfileId,
qualityProfileId: radarrSettings.activeProfileId,
rootFolderPath: radarrSettings.activeDirectory,
Expand Down Expand Up @@ -212,7 +213,8 @@ export class MediaRequest {
throw new Error('Series was missing tvdb id');
}

await sonarr.addSeries({
// Run this asynchronously so we don't wait for it on the UI side
sonarr.addSeries({
profileId: sonarrSettings.activeProfileId,
rootFolderPath: sonarrSettings.activeDirectory,
title: series.name,
Expand Down
2 changes: 2 additions & 0 deletions server/routes/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ import discoverRoutes from './discover';
import requestRoutes from './request';
import movieRoutes from './movie';
import tvRoutes from './tv';
import mediaRoutes from './media';

const router = Router();

Expand All @@ -25,6 +26,7 @@ router.use('/discover', isAuthenticated(), discoverRoutes);
router.use('/request', isAuthenticated(), requestRoutes);
router.use('/movie', isAuthenticated(), movieRoutes);
router.use('/tv', isAuthenticated(), tvRoutes);
router.use('/media', isAuthenticated(), mediaRoutes);
router.use('/auth', authRoutes);

router.get('/settings/public', (_req, res) => {
Expand Down
71 changes: 71 additions & 0 deletions server/routes/media.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,71 @@
import { Router } from 'express';
import { getRepository, FindOperator } from 'typeorm';
import Media from '../entity/Media';
import { MediaStatus } from '../constants/media';

export interface MediaResultsResponse {
pageInfo: {
pages: number;
page: number;
results: number;
pageSize: number;
};
results: Media[];
}

const mediaRoutes = Router();

mediaRoutes.get('/', async (req, res, next) => {
const mediaRepository = getRepository(Media);

const pageSize = Number(req.query.take) ?? 20;
const skip = Number(req.query.skip) ?? 0;

let statusFilter:
| MediaStatus
| FindOperator<MediaStatus>
| undefined = undefined;

switch (req.query.filter) {
case 'available':
statusFilter = MediaStatus.AVAILABLE;
break;
case 'partial':
statusFilter = MediaStatus.PARTIALLY_AVAILABLE;
break;
case 'processing':
statusFilter = MediaStatus.PROCESSING;
break;
case 'pending':
statusFilter = MediaStatus.PENDING;
break;
default:
statusFilter = undefined;
}

try {
const [media, mediaCount] = await mediaRepository.findAndCount({
order: {
id: 'DESC',
},
where: {
status: statusFilter,
},
take: pageSize,
skip,
});
return res.status(200).json({
pageInfo: {
pages: Math.ceil(mediaCount / pageSize),
pageSize,
results: mediaCount,
page: Math.ceil(skip / pageSize) + 1,
},
results: media,
} as MediaResultsResponse);
} catch (e) {
next({ status: 500, message: e.message });
}
});

export default mediaRoutes;
6 changes: 4 additions & 2 deletions server/routes/request.ts
Original file line number Diff line number Diff line change
Expand Up @@ -20,15 +20,17 @@ requestRoutes.get('/', async (req, res, next) => {
id: 'DESC',
},
relations: ['media'],
take: 20,
take: Number(req.query.take) ?? 20,
skip: Number(req.query.skip) ?? 0,
})
: await requestRepository.find({
where: { requestedBy: { id: req.user?.id } },
relations: ['media'],
order: {
id: 'DESC',
},
take: 20,
take: Number(req.query.limit) ?? 20,
skip: Number(req.query.skip) ?? 0,
});

return res.status(200).json(requests);
Expand Down
49 changes: 48 additions & 1 deletion src/components/Discover/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -6,13 +6,17 @@ import { MediaRequest } from '../../../server/entity/MediaRequest';
import RequestCard from '../TitleCard/RequestCard';
import Slider from '../Slider';
import Link from 'next/link';
import { defineMessages, FormattedMessage } from 'react-intl';
import { defineMessages, FormattedMessage, useIntl } from 'react-intl';
import { LanguageContext } from '../../context/LanguageContext';
import type Media from '../../../server/entity/Media';
import type { MediaResultsResponse } from '../../../server/routes/media';

const messages = defineMessages({
recentrequests: 'Recent Requests',
popularmovies: 'Popular Movies',
populartv: 'Popular Series',
recentlyAdded: 'Recently Added',
nopending: 'No Pending Requests',
});

interface MovieDiscoverResult {
Expand All @@ -30,6 +34,7 @@ interface TvDiscoverResult {
}

const Discover: React.FC = () => {
const intl = useIntl();
const { locale } = useContext(LanguageContext);
const { data: movieData, error: movieError } = useSWR<MovieDiscoverResult>(
`/api/v1/discover/movies?language=${locale}`
Expand All @@ -38,12 +43,53 @@ const Discover: React.FC = () => {
`/api/v1/discover/tv?language=${locale}`
);

const { data: media, error: mediaError } = useSWR<MediaResultsResponse>(
'/api/v1/media?filter=available&take=20'
);

const { data: requests, error: requestError } = useSWR<MediaRequest[]>(
'/api/v1/request'
);

return (
<>
<div className="md:flex md:items-center md:justify-between mb-4 mt-6">
<div className="flex-1 min-w-0">
<Link href="/recent">
<a className="inline-flex text-xl leading-7 text-cool-gray-300 hover:text-white sm:text-2xl sm:leading-9 sm:truncate items-center">
<span>
<FormattedMessage {...messages.recentlyAdded} />
</span>
<svg
className="w-6 h-6 ml-2"
fill="none"
stroke="currentColor"
viewBox="0 0 24 24"
xmlns="http://www.w3.org/2000/svg"
>
<path
strokeLinecap="round"
strokeLinejoin="round"
strokeWidth={2}
d="M13 9l3 3m0 0l-3 3m3-3H8m13 0a9 9 0 11-18 0 9 9 0 0118 0z"
/>
</svg>
</a>
</Link>
</div>
</div>
<Slider
sliderKey="requests"
isLoading={!media && !mediaError}
isEmpty={!!media && !mediaError && media.results.length === 0}
items={media?.results?.map((item) => (
<RequestCard
key={`media-slider-item-${item.id}`}
tmdbId={item.tmdbId}
type={item.mediaType}
/>
))}
/>
<div className="md:flex md:items-center md:justify-between mb-4 mt-6">
<div className="flex-1 min-w-0">
<Link href="/requests">
Expand Down Expand Up @@ -80,6 +126,7 @@ const Discover: React.FC = () => {
type={request.media.mediaType}
/>
))}
emptyMessage={intl.formatMessage(messages.nopending)}
/>
<div className="md:flex md:items-center md:justify-between mb-4 mt-6">
<div className="flex-1 min-w-0">
Expand Down
31 changes: 16 additions & 15 deletions src/components/RequestModal/MovieRequestModal.tsx
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import React, { useCallback } from 'react';
import React, { useCallback, useState, useEffect } from 'react';
import Modal from '../Common/Modal';
import { useUser } from '../../hooks/useUser';
import { Permission } from '../../../server/lib/permissions';
Expand Down Expand Up @@ -35,17 +35,22 @@ const MovieRequestModal: React.FC<RequestModalProps> = ({
onUpdating,
...props
}) => {
const [isUpdating, setIsUpdating] = useState(false);
const { addToast } = useToasts();
const { data, error } = useSWR<MovieDetails>(`/api/v1/movie/${tmdbId}`, {
revalidateOnMount: true,
});
const intl = useIntl();
const { user, hasPermission } = useUser();

const sendRequest = useCallback(async () => {
useEffect(() => {
if (onUpdating) {
onUpdating(true);
onUpdating(isUpdating);
}
}, [isUpdating, onUpdating]);

const sendRequest = useCallback(async () => {
setIsUpdating(true);
const response = await axios.post<MediaRequest>('/api/v1/request', {
mediaId: data?.id,
mediaType: 'movie',
Expand All @@ -61,18 +66,14 @@ const MovieRequestModal: React.FC<RequestModalProps> = ({
</span>,
{ appearance: 'success', autoDismiss: true }
);
if (onUpdating) {
onUpdating(false);
}
setIsUpdating(false);
}
}, [data, onComplete, onUpdating, addToast]);
}, [data, onComplete, addToast]);

const activeRequest = data?.mediaInfo?.requests?.[0];

const cancelRequest = async () => {
if (onUpdating) {
onUpdating(true);
}
setIsUpdating(true);
const response = await axios.delete<MediaRequest>(
`/api/v1/request/${activeRequest?.id}`
);
Expand All @@ -87,9 +88,7 @@ const MovieRequestModal: React.FC<RequestModalProps> = ({
</span>,
{ appearance: 'success', autoDismiss: true }
);
if (onUpdating) {
onUpdating(false);
}
setIsUpdating(false);
}
};

Expand All @@ -109,8 +108,9 @@ const MovieRequestModal: React.FC<RequestModalProps> = ({
backgroundClickable
onCancel={onCancel}
onOk={isOwner ? () => cancelRequest() : undefined}
okDisabled={isUpdating}
title={`Pending request for ${data?.title}`}
okText={'Cancel Request'}
okText={isUpdating ? 'Cancelling...' : 'Cancel Request'}
okButtonType={'danger'}
cancelText="Close"
iconSvg={<DownloadIcon className="w-6 h-6" />}
Expand All @@ -128,8 +128,9 @@ const MovieRequestModal: React.FC<RequestModalProps> = ({
backgroundClickable
onCancel={onCancel}
onOk={sendRequest}
okDisabled={isUpdating}
title={`Request ${data?.title}`}
okText={'Request'}
okText={isUpdating ? 'Requesting...' : 'Request'}
okButtonType={'primary'}
iconSvg={<DownloadIcon className="w-6 h-6" />}
{...props}
Expand Down

0 comments on commit 06dc606

Please sign in to comment.