From c7b361bcac657832c368991fd550cb1aee85a042 Mon Sep 17 00:00:00 2001
From: mrjvs <jellevs@gmail.com>
Date: Fri, 1 Dec 2023 23:29:24 +0100
Subject: [PATCH 1/5] add hover to locale dropdown

---
 src/components/form/Dropdown.tsx | 2 +-
 themes/default.ts                | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/components/form/Dropdown.tsx b/src/components/form/Dropdown.tsx
index c1e36c01..febe923f 100644
--- a/src/components/form/Dropdown.tsx
+++ b/src/components/form/Dropdown.tsx
@@ -21,7 +21,7 @@ export function Dropdown(props: DropdownProps) {
       <Listbox value={props.selectedItem} onChange={props.setSelectedItem}>
         {() => (
           <>
-            <Listbox.Button className="relative w-full rounded-lg bg-dropdown-background py-3 pl-3 pr-10 text-left text-white shadow-md focus:outline-none tabbable cursor-pointer">
+            <Listbox.Button className="relative w-full rounded-lg bg-dropdown-background hover:bg-dropdown-hoverBackground py-3 pl-3 pr-10 text-left text-white shadow-md focus:outline-none tabbable cursor-pointer">
               <span className="flex gap-4 items-center truncate">
                 {props.selectedItem.leftIcon
                   ? props.selectedItem.leftIcon
diff --git a/themes/default.ts b/themes/default.ts
index f0dd5113..bc855bc4 100644
--- a/themes/default.ts
+++ b/themes/default.ts
@@ -97,6 +97,7 @@ export const defaultTheme = {
       dropdown: {
         background: "#171728",
         altBackground: "#151525",
+        hoverBackground: "#202036",
         highlight: "#afa349",
         highlightHover: "#FCEC61",
         text: "#846D95",

From 83bc9637b0d1561c9f930e9da66bd29e385826ee Mon Sep 17 00:00:00 2001
From: mrjvs <jellevs@gmail.com>
Date: Fri, 1 Dec 2023 23:36:48 +0100
Subject: [PATCH 2/5] Prevent language duplication in search

---
 src/components/player/atoms/settings/CaptionsView.tsx | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/components/player/atoms/settings/CaptionsView.tsx b/src/components/player/atoms/settings/CaptionsView.tsx
index 437d5fde..bfee8b5f 100644
--- a/src/components/player/atoms/settings/CaptionsView.tsx
+++ b/src/components/player/atoms/settings/CaptionsView.tsx
@@ -130,10 +130,12 @@ export function CaptionsView({ id }: { id: string }) {
     [selectLanguage, setCurrentlyDownloading]
   );
 
-  const content = subtitleList.map((v) => {
+  const content = subtitleList.map((v, i) => {
     return (
       <CaptionOption
-        key={v.language}
+        // key must use index to prevent url collisions
+        // eslint-disable-next-line react/no-array-index-key
+        key={`${i}-${v.url}`}
         countryCode={v.language}
         selected={lang === v.language}
         loading={v.language === currentlyDownloading && downloadReq.loading}

From 7841fadcb6050601b865803f4844ae653b39caee Mon Sep 17 00:00:00 2001
From: mrjvs <jellevs@gmail.com>
Date: Sat, 2 Dec 2023 00:02:09 +0100
Subject: [PATCH 3/5] make email an optional setting + add device name
 validation

---
 src/assets/locales/en.json                 |  1 +
 src/components/layout/Footer.tsx           |  3 +++
 src/pages/Dmca.tsx                         |  8 ++++++--
 src/pages/parts/auth/AccountCreatePart.tsx | 16 ++++++++++++++--
 src/pages/parts/auth/LoginFormPart.tsx     |  9 ++++++---
 src/setup/App.tsx                          |  7 +++++--
 src/setup/config.ts                        |  5 +++++
 7 files changed, 40 insertions(+), 9 deletions(-)

diff --git a/src/assets/locales/en.json b/src/assets/locales/en.json
index 78653fed..d4adb9a1 100644
--- a/src/assets/locales/en.json
+++ b/src/assets/locales/en.json
@@ -16,6 +16,7 @@
       "title": "Login to your account",
       "description": "Please enter your passphrase to login to your account",
       "validationError": "Invalid or incomplete passphrase",
+      "deviceLengthError": "Please entire a device name",
       "submit": "Login",
       "passphraseLabel": "12-Word passphrase",
       "passphrasePlaceholder": "Passphrase"
diff --git a/src/components/layout/Footer.tsx b/src/components/layout/Footer.tsx
index 3704c446..ec4951a6 100644
--- a/src/components/layout/Footer.tsx
+++ b/src/components/layout/Footer.tsx
@@ -4,6 +4,7 @@ import { useHistory } from "react-router-dom";
 import { Icon, Icons } from "@/components/Icon";
 import { BrandPill } from "@/components/layout/BrandPill";
 import { WideContainer } from "@/components/layout/WideContainer";
+import { shouldHaveDmcaPage } from "@/pages/Dmca";
 import { conf } from "@/setup/config";
 
 function FooterLink(props: {
@@ -30,6 +31,8 @@ function Dmca() {
   const { t } = useTranslation();
   const history = useHistory();
 
+  if (!shouldHaveDmcaPage()) return null;
+
   return (
     <FooterLink icon={Icons.DRAGON} onClick={() => history.push("/dmca")}>
       {t("footer.links.dmca")}
diff --git a/src/pages/Dmca.tsx b/src/pages/Dmca.tsx
index 4aff3985..d0ceebd3 100644
--- a/src/pages/Dmca.tsx
+++ b/src/pages/Dmca.tsx
@@ -4,10 +4,14 @@ import { Icon, Icons } from "@/components/Icon";
 import { ThinContainer } from "@/components/layout/ThinContainer";
 import { Heading1, Paragraph } from "@/components/utils/Text";
 import { PageTitle } from "@/pages/parts/util/PageTitle";
+import { conf } from "@/setup/config";
 
 import { SubPageLayout } from "./layouts/SubPageLayout";
 
-// TODO make email a constant
+export function shouldHaveDmcaPage() {
+  return !!conf().DMCA_EMAIL;
+}
+
 export function DmcaPage() {
   const { t } = useTranslation();
 
@@ -19,7 +23,7 @@ export function DmcaPage() {
         <Paragraph>{t("screens.dmca.text")}</Paragraph>
         <Paragraph className="flex space-x-3 items-center">
           <Icon icon={Icons.MAIL} />
-          <span>dmca@movie-web.app</span>
+          <span>{conf().DMCA_EMAIL ?? ""}</span>
         </Paragraph>
       </ThinContainer>
     </SubPageLayout>
diff --git a/src/pages/parts/auth/AccountCreatePart.tsx b/src/pages/parts/auth/AccountCreatePart.tsx
index 3de705f7..7fa90ff7 100644
--- a/src/pages/parts/auth/AccountCreatePart.tsx
+++ b/src/pages/parts/auth/AccountCreatePart.tsx
@@ -32,11 +32,18 @@ export function AccountCreatePart(props: AccountCreatePartProps) {
   const [colorB, setColorB] = useState("#2E65CF");
   const [userIcon, setUserIcon] = useState<UserIcons>(UserIcons.USER);
   const { t } = useTranslation();
-  // TODO validate device and account before next step
+  const [hasDeviceError, setHasDeviceError] = useState(false);
 
   const nextStep = useCallback(() => {
+    setHasDeviceError(false);
+    const validatedDevice = device.trim();
+    if (validatedDevice.length === 0) {
+      setHasDeviceError(true);
+      return;
+    }
+
     props.onNext?.({
-      device,
+      device: validatedDevice,
       profile: {
         colorA,
         colorB,
@@ -75,6 +82,11 @@ export function AccountCreatePart(props: AccountCreatePartProps) {
           value={userIcon}
           onInput={setUserIcon}
         />
+        {hasDeviceError ? (
+          <p className="text-authentication-errorText">
+            {t("auth.login.deviceLengthError")}
+          </p>
+        ) : null}
       </div>
       <LargeCardButtons>
         <Button theme="purple" onClick={() => nextStep()}>
diff --git a/src/pages/parts/auth/LoginFormPart.tsx b/src/pages/parts/auth/LoginFormPart.tsx
index 16cec00b..82836e74 100644
--- a/src/pages/parts/auth/LoginFormPart.tsx
+++ b/src/pages/parts/auth/LoginFormPart.tsx
@@ -29,14 +29,17 @@ export function LoginFormPart(props: LoginFormPartProps) {
 
   const [result, execute] = useAsyncFn(
     async (inputMnemonic: string, inputdevice: string) => {
-      // TODO verify valid device input
       if (!verifyValidMnemonic(inputMnemonic))
         throw new Error(t("auth.login.validationError") ?? undefined);
 
+      const validatedDevice = inputdevice.trim();
+      if (validatedDevice.length === 0)
+        throw new Error(t("auth.login.deviceLengthError") ?? undefined);
+
       const account = await login({
         mnemonic: inputMnemonic,
         userData: {
-          device: inputdevice,
+          device: validatedDevice,
         },
       });
 
@@ -46,7 +49,7 @@ export function LoginFormPart(props: LoginFormPartProps) {
 
       props.onLogin?.();
     },
-    [props, login, restore]
+    [props, login, restore, t]
   );
 
   return (
diff --git a/src/setup/App.tsx b/src/setup/App.tsx
index 03b1e69c..cebb464a 100644
--- a/src/setup/App.tsx
+++ b/src/setup/App.tsx
@@ -14,7 +14,7 @@ import { useOnlineListener } from "@/hooks/usePing";
 import { AboutPage } from "@/pages/About";
 import { AdminPage } from "@/pages/admin/AdminPage";
 import VideoTesterView from "@/pages/developer/VideoTesterView";
-import { DmcaPage } from "@/pages/Dmca";
+import { DmcaPage, shouldHaveDmcaPage } from "@/pages/Dmca";
 import { NotFoundPage } from "@/pages/errors/NotFoundPage";
 import { HomePage } from "@/pages/HomePage";
 import { LoginPage } from "@/pages/Login";
@@ -93,7 +93,10 @@ function App() {
         <Route exact path="/register" component={RegisterPage} />
         <Route exact path="/login" component={LoginPage} />
         <Route exact path="/faq" component={AboutPage} />
-        <Route exact path="/dmca" component={DmcaPage} />
+
+        {shouldHaveDmcaPage() ? (
+          <Route exact path="/dmca" component={DmcaPage} />
+        ) : null}
 
         {/* Settings page */}
         <Route exact path="/settings" component={SettingsPage} />
diff --git a/src/setup/config.ts b/src/setup/config.ts
index 1a908174..5402ca6b 100644
--- a/src/setup/config.ts
+++ b/src/setup/config.ts
@@ -10,6 +10,7 @@ interface Config {
   GITHUB_LINK: string;
   DONATION_LINK: string;
   DISCORD_LINK: string;
+  DMCA_EMAIL: string;
   TMDB_READ_API_KEY: string;
   CORS_PROXY_URL: string;
   NORMAL_ROUTER: boolean;
@@ -22,6 +23,7 @@ export interface RuntimeConfig {
   GITHUB_LINK: string;
   DONATION_LINK: string;
   DISCORD_LINK: string;
+  DMCA_EMAIL: string | null;
   TMDB_READ_API_KEY: string;
   NORMAL_ROUTER: boolean;
   PROXY_URLS: string[];
@@ -35,6 +37,7 @@ const env: Record<keyof Config, undefined | string> = {
   GITHUB_LINK: undefined,
   DONATION_LINK: undefined,
   DISCORD_LINK: undefined,
+  DMCA_EMAIL: import.meta.env.VITE_DMCA_EMAIL,
   CORS_PROXY_URL: import.meta.env.VITE_CORS_PROXY_URL,
   NORMAL_ROUTER: import.meta.env.VITE_NORMAL_ROUTER,
   BACKEND_URL: import.meta.env.VITE_BACKEND_URL,
@@ -54,11 +57,13 @@ function getKey(key: keyof Config, defaultString?: string): string {
 }
 
 export function conf(): RuntimeConfig {
+  const dmcaEmail = getKey("DMCA_EMAIL");
   return {
     APP_VERSION,
     GITHUB_LINK,
     DONATION_LINK,
     DISCORD_LINK,
+    DMCA_EMAIL: dmcaEmail.length > 0 ? dmcaEmail : null,
     BACKEND_URL: getKey("BACKEND_URL"),
     TMDB_READ_API_KEY: getKey("TMDB_READ_API_KEY"),
     PROXY_URLS: getKey("CORS_PROXY_URL")

From 29d0b05845ecea698d829d91f436a81d6fc74722 Mon Sep 17 00:00:00 2001
From: mrjvs <jellevs@gmail.com>
Date: Sat, 2 Dec 2023 00:13:48 +0100
Subject: [PATCH 4/5] remove buffered todo, after research it should already be
 working as intended

---
 src/components/player/utils/handleBuffered.ts | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/components/player/utils/handleBuffered.ts b/src/components/player/utils/handleBuffered.ts
index 6379d83c..ee19ae6a 100644
--- a/src/components/player/utils/handleBuffered.ts
+++ b/src/components/player/utils/handleBuffered.ts
@@ -1,5 +1,4 @@
 export function handleBuffered(time: number, buffered: TimeRanges): number {
-  // TODO normalize the buffer sections into one section. they can be stitched together
   for (let i = 0; i < buffered.length; i += 1) {
     if (buffered.start(buffered.length - 1 - i) < time) {
       return buffered.end(buffered.length - 1 - i);

From 59cc8b2bc4f322020b9aa054ba9dd7f6bd13329a Mon Sep 17 00:00:00 2001
From: mrjvs <jellevs@gmail.com>
Date: Sat, 2 Dec 2023 12:57:37 +0100
Subject: [PATCH 5/5] Update src/assets/locales/en.json

Co-authored-by: William Oldham <github@binaryoverload.co.uk>
---
 src/assets/locales/en.json | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/assets/locales/en.json b/src/assets/locales/en.json
index d4adb9a1..8d9b0b72 100644
--- a/src/assets/locales/en.json
+++ b/src/assets/locales/en.json
@@ -16,7 +16,7 @@
       "title": "Login to your account",
       "description": "Please enter your passphrase to login to your account",
       "validationError": "Invalid or incomplete passphrase",
-      "deviceLengthError": "Please entire a device name",
+      "deviceLengthError": "Please enter a device name",
       "submit": "Login",
       "passphraseLabel": "12-Word passphrase",
       "passphrasePlaceholder": "Passphrase"