fix: fix a few misc unused imports and useless assignments/conditionals

pull/182/head
sct 4 years ago
parent 5205ece022
commit 8e6daf7bd2

@ -522,7 +522,7 @@ const MovieDetails: React.FC<MovieDetailsProps> = ({ movie }) => {
</div>
<Slider
sliderKey="cast"
isLoading={!data && !error}
isLoading={false}
isEmpty={false}
items={data?.credits.cast.slice(0, 20).map((person) => (
<PersonCard

@ -473,7 +473,7 @@ const TvDetails: React.FC<TvDetailsProps> = ({ tv }) => {
</div>
<Slider
sliderKey="cast"
isLoading={!data && !error}
isLoading={false}
isEmpty={false}
items={data?.credits.cast.slice(0, 20).map((person) => (
<PersonCard

@ -1,4 +1,4 @@
import React, { useEffect } from 'react';
import React, { useEffect, useRef } from 'react';
import { User, useUser } from '../hooks/useUser';
import { useRouter } from 'next/dist/client/router';
@ -17,20 +17,19 @@ export const UserContext: React.FC<UserContextProps> = ({
}) => {
const { user, error, revalidate } = useUser({ initialData: initialUser });
const router = useRouter();
const routing = useRef(false);
useEffect(() => {
revalidate();
}, [router.pathname, revalidate]);
useEffect(() => {
let routing = false;
if (
!router.pathname.match(/(setup|login)/) &&
(!user || error) &&
!routing
!routing.current
) {
routing = true;
routing.current = true;
location.href = '/login';
}
}, [router, user, error]);

@ -100,13 +100,11 @@ CoreApp.getInitialProps = async (initialProps) => {
if (ctx.res) {
// Check if app is initialized and redirect if necessary
let initialized = true;
const response = await axios.get<{ initialized: boolean }>(
`http://localhost:${process.env.PORT || 3000}/api/v1/settings/public`
);
initialized = response.data.initialized;
const initialized = response.data.initialized;
if (!initialized) {
if (!router.pathname.match(/(setup|login\/plex)/)) {

Loading…
Cancel
Save