From 29270121efd12f9f5555a5a06f84bfd3b54aaa85 Mon Sep 17 00:00:00 2001
From: duncte123 <contact@duncte123.me>
Date: Mon, 26 Feb 2024 09:09:47 +0100
Subject: [PATCH 1/2] Use new oengus display names and support the youtube
 connection

---
 .eslintrc.browser.js                                  |  4 ++--
 .eslintrc.extension.js                                |  6 ++++--
 .gitignore                                            |  1 +
 configschema.json                                     |  7 +------
 schemas/reused/RunData.json                           |  5 ++++-
 src/dashboard/oengus-schedule-import/main.vue         | 11 -----------
 .../run-modification-dialog/components/Player.vue     |  8 ++++++++
 src/extension/oengus-import.ts                        | 11 ++++++-----
 src/types/Oengus.d.ts                                 |  5 +++--
 src/types/RunData.d.ts                                |  1 +
 src/types/schemas/configschema.d.ts                   |  1 -
 src/types/schemas/runDataActiveRun.d.ts               |  1 +
 src/types/schemas/runDataArray.d.ts                   |  1 +
 webpack.config.mjs                                    |  2 +-
 14 files changed, 33 insertions(+), 31 deletions(-)

diff --git a/.eslintrc.browser.js b/.eslintrc.browser.js
index c1678a13..61e25cbc 100644
--- a/.eslintrc.browser.js
+++ b/.eslintrc.browser.js
@@ -8,7 +8,7 @@ module.exports = {
   parser: 'vue-eslint-parser',
   parserOptions: {
     parser: '@typescript-eslint/parser',
-    project: 'tsconfig.browser.json',
+    project: path.join(__dirname, 'tsconfig.browser.json'),
     extraFileExtensions: ['.vue'],
     ecmaVersion: 2020,
   },
@@ -31,7 +31,7 @@ module.exports = {
     'import/resolver': {
       typescript: {
         // This is needed to properly resolve paths.
-        project: 'tsconfig.browser.json',
+        project: path.join(__dirname, 'tsconfig.browser.json'),
       },
       webpack: {
         config: path.join(__dirname, 'webpack.config.mjs'),
diff --git a/.eslintrc.extension.js b/.eslintrc.extension.js
index a7f7334a..3f6a4a9f 100644
--- a/.eslintrc.extension.js
+++ b/.eslintrc.extension.js
@@ -1,3 +1,5 @@
+const path = require('path');
+
 module.exports = {
   root: true,
   env: {
@@ -5,7 +7,7 @@ module.exports = {
   },
   parser: '@typescript-eslint/parser',
   parserOptions: {
-    project: 'tsconfig.extension.json',
+    project: path.join(__dirname, 'tsconfig.extension.json'),
   },
   plugins: [
     '@typescript-eslint',
@@ -21,7 +23,7 @@ module.exports = {
     'import/resolver': {
       typescript: {
         // This is needed to properly resolve paths.
-        project: 'tsconfig.extension.json',
+        project: path.join(__dirname, 'tsconfig.extension.json'),
       },
     },
     'import/extensions': ['.js', '.jsx', '.ts', '.tsx'],
diff --git a/.gitignore b/.gitignore
index be54081b..24f6661e 100644
--- a/.gitignore
+++ b/.gitignore
@@ -133,3 +133,4 @@ dist
 /dashboard/
 /extension/
 /graphics/
+.idea/
diff --git a/configschema.json b/configschema.json
index 35c8d041..51ffb03b 100644
--- a/configschema.json
+++ b/configschema.json
@@ -193,10 +193,6 @@
           "type": "string",
           "default": "SHORTNAME"
         },
-        "useJapanese": {
-          "type": "boolean",
-          "default": false
-        },
         "ignoreGamesWhileImporting": {
           "$ref": "#/definitions/ignoreGamesWhileImporting"
         },
@@ -217,7 +213,6 @@
       },
       "required": [
         "defaultMarathon",
-        "useJapanese",
         "disableSpeedrunComLookup"
       ]
     },
@@ -271,4 +266,4 @@
     "horaro",
     "oengus"
   ]
-}
\ No newline at end of file
+}
diff --git a/schemas/reused/RunData.json b/schemas/reused/RunData.json
index e589e778..0f812d61 100644
--- a/schemas/reused/RunData.json
+++ b/schemas/reused/RunData.json
@@ -86,6 +86,9 @@
                   "properties": {
                     "twitch": {
                       "type": "string"
+                    },
+                    "youtube": {
+                      "type": "string"
                     }
                   }
                 },
@@ -136,4 +139,4 @@
     "customData",
     "id"
   ]
-}
\ No newline at end of file
+}
diff --git a/src/dashboard/oengus-schedule-import/main.vue b/src/dashboard/oengus-schedule-import/main.vue
index cbe603c8..6bc1b591 100644
--- a/src/dashboard/oengus-schedule-import/main.vue
+++ b/src/dashboard/oengus-schedule-import/main.vue
@@ -5,7 +5,6 @@
     "shortname": "Oengus Marathon Shortname",
     "helpText": "Insert the Oengus marathon shortname (not including \"/schedule\") above and press the \"Import Schedule Data\" button.",
     "importInProgressHelpText": "Import currently in progress...",
-    "useJapaneseNames": "Use Japanese names?",
     "import": "Import Schedule Data",
     "importProgress": "Importing {item}/{total}"
   },
@@ -14,7 +13,6 @@
     "shortname": "Oengusマラソンの略称",
     "helpText": "上記にインポートしたいOengusのイベントの略称を入力し(\"/schedule\"を含めないでください)、「スケジュール情報のインポート」ボタンを押してください。",
     "importInProgressHelpText": "インポート処理の実行中...",
-    "useJapaneseNames": "日本語ユーザーネームを使用しますか？",
     "import": "スケジュール情報のインポート",
     "importProgress": "{item}/{total}件をインポート"
   }
@@ -38,13 +36,6 @@
         <div>
           {{ $t('helpText') }}
         </div>
-        <!-- Switch use Japanese or not for importing data -->
-        <v-switch
-          v-model="useJapanese"
-          class="ma-1"
-          hide-details
-          :label="$t('useJapaneseNames')"
-        />
       </template>
       <template v-else>
         {{ $t('importInProgressHelpText') }}
@@ -82,7 +73,6 @@ import { replicantNS } from '../_misc/replicant_store';
 export default class extends Vue {
   @replicantNS.State((s) => s.reps.oengusImportStatus) readonly importStatus!: OengusImportStatus;
   marathonShort = nodecg.bundleConfig.oengus.defaultMarathon || '';
-  useJapanese = nodecg.bundleConfig.oengus.useJapanese;
 
   importConfirm(): void {
     const dialog = getDialog('alert-dialog') as Alert.Dialog;
@@ -99,7 +89,6 @@ export default class extends Vue {
       try {
         await nodecg.sendMessage('importOengusSchedule', {
           marathonShort: this.marathonShort,
-          useJapanese: this.useJapanese,
         });
       } catch (err) {
         // catch
diff --git a/src/dashboard/run-modification-dialog/components/Player.vue b/src/dashboard/run-modification-dialog/components/Player.vue
index 2087c65e..6392b2f4 100644
--- a/src/dashboard/run-modification-dialog/components/Player.vue
+++ b/src/dashboard/run-modification-dialog/components/Player.vue
@@ -3,6 +3,7 @@
   "en": {
     "name": "Name",
     "twitch": "Twitch",
+    "youtube": "YouTube",
     "countryCode": "Country Code",
     "removePlayer": "Remove Player",
     "pronouns": "Pronouns"
@@ -10,6 +11,7 @@
   "ja": {
     "name": "プレイヤー名",
     "twitch": "Twitch ID",
+    "youtube": "YouTube",
     "countryCode": "国名コード",
     "removePlayer": "プレイヤーの削除",
     "pronouns": "Pronouns"
@@ -33,6 +35,12 @@
       :label="$t('twitch')"
       left-border
     />
+    <text-input
+      :value="playerData.social.youtube"
+      @input="updatePlayerDataProp('social.youtube', $event)"
+      :label="$t('youtube')"
+      left-border
+    />
     <text-input
       :value="playerData.country"
       @input="updatePlayerDataProp('country', $event)"
diff --git a/src/extension/oengus-import.ts b/src/extension/oengus-import.ts
index 724b91cd..a464ddef 100644
--- a/src/extension/oengus-import.ts
+++ b/src/extension/oengus-import.ts
@@ -82,9 +82,8 @@ function resetImportStatus(): void {
 /**
  * Import schedule data in from Oengus.
  * @param marathonShort Oengus' marathon shortname you want to import.
- * @param useJapanese If you want to use usernameJapanese from the user data.
  */
-async function importSchedule(marathonShort: string, useJapanese: boolean): Promise<void> {
+async function importSchedule(marathonShort: string): Promise<void> {
   try {
     oengusImportStatus.value.importing = true;
     const marathonResp = await get(`/marathons/${marathonShort}`);
@@ -182,16 +181,18 @@ async function importSchedule(marathonShort: string, useJapanese: boolean): Prom
         };
         const playerTwitch = runner.connections
           ?.find((c) => c.platform === 'TWITCH')?.username || runner.twitchName;
+        const playerYoutube = runner.connections
+          ?.find((c) => c.platform === 'YOUTUBE')?.username;
         const playerPronouns = typeof runner.pronouns === 'string'
           ? runner.pronouns.split(',')
           : runner.pronouns;
         const player: RunDataPlayer = {
-          name: (useJapanese && runner.usernameJapanese)
-            ? runner.usernameJapanese : runner.username,
+          name: runner.displayName || runner.username,
           id: uuid(),
           teamID: team.id,
           social: {
             twitch: playerTwitch || undefined,
+            youtube: playerYoutube || undefined,
           },
           country: runner.country?.toLowerCase() || undefined,
           pronouns: playerPronouns?.join(', ') || undefined,
@@ -240,7 +241,7 @@ nodecg.listenFor('importOengusSchedule', async (data, ack) => {
       throw new Error('Already importing schedule');
     }
     nodecg.log.info('[Oengus Import] Started importing schedule');
-    await importSchedule(data.marathonShort, data.useJapanese);
+    await importSchedule(data.marathonShort);
     nodecg.log.info('[Oengus Import] Successfully imported schedule from Oengus');
     processAck(ack, null);
   } catch (err) {
diff --git a/src/types/Oengus.d.ts b/src/types/Oengus.d.ts
index 3c6f8f50..003f8684 100644
--- a/src/types/Oengus.d.ts
+++ b/src/types/Oengus.d.ts
@@ -37,7 +37,8 @@ export enum OengusRunType {
 export interface OengusUser {
   id: number;
   username: string;
-  usernameJapanese: string | null;
+  displayName: string;
+  usernameJapanese: string | null; // deprecated
   enabled: boolean;
   twitterName?: string | null; // deprecated
   twitchName?: string | null; // deprecated
@@ -49,7 +50,7 @@ export interface OengusUser {
 
 export interface OengusUserConnections {
   id: number;
-  platform: 'DISCORD' | 'SPEEDRUNCOM' | 'TWITCH' | 'TWITTER';
+  platform: 'DISCORD' | 'SPEEDRUNCOM' | 'TWITCH' | 'TWITTER' | 'YOUTUBE';
   username: string;
   usernameValidForPlatform: boolean;
 }
diff --git a/src/types/RunData.d.ts b/src/types/RunData.d.ts
index 72697c74..8bb35166 100644
--- a/src/types/RunData.d.ts
+++ b/src/types/RunData.d.ts
@@ -35,6 +35,7 @@ export interface RunDataPlayer {
   pronouns?: string;
   social: {
     twitch?: string;
+    youtube?: string;
   };
   customData: {
     [key: string]: string;
diff --git a/src/types/schemas/configschema.d.ts b/src/types/schemas/configschema.d.ts
index 9ec79980..c98a7322 100644
--- a/src/types/schemas/configschema.d.ts
+++ b/src/types/schemas/configschema.d.ts
@@ -37,7 +37,6 @@ export interface Configschema {
 	horaro: Horaro;
 	oengus: {
 		defaultMarathon: string;
-		useJapanese: boolean;
 		ignoreGamesWhileImporting?: IgnoreGamesWhileImporting;
 		disableSpeedrunComLookup: boolean;
 		useSandbox?: boolean | null;
diff --git a/src/types/schemas/runDataActiveRun.d.ts b/src/types/schemas/runDataActiveRun.d.ts
index a2e9bfe7..7639102f 100644
--- a/src/types/schemas/runDataActiveRun.d.ts
+++ b/src/types/schemas/runDataActiveRun.d.ts
@@ -33,6 +33,7 @@ export interface RunData {
 			pronouns?: string;
 			social: {
 				twitch?: string;
+				youtube?: string;
 			};
 			customData: {
 				[k: string]: string;
diff --git a/src/types/schemas/runDataArray.d.ts b/src/types/schemas/runDataArray.d.ts
index fef67a0c..da3b603c 100644
--- a/src/types/schemas/runDataArray.d.ts
+++ b/src/types/schemas/runDataArray.d.ts
@@ -33,6 +33,7 @@ export interface RunData {
 			pronouns?: string;
 			social: {
 				twitch?: string;
+				youtube?: string;
 			};
 			customData: {
 				[k: string]: string;
diff --git a/webpack.config.mjs b/webpack.config.mjs
index 0c195de2..d20d95b9 100644
--- a/webpack.config.mjs
+++ b/webpack.config.mjs
@@ -81,7 +81,7 @@ const config = (name) => {
       },
       plugins: [
         new TsConfigPathsPlugin({
-          configFile: 'tsconfig.browser.json',
+          configFile: path.resolve(__dirname, 'tsconfig.browser.json'),
         }),
       ],
     },

From 1c4877d0eec7d62d3201c18a1577137fc480d310 Mon Sep 17 00:00:00 2001
From: zoton2 <zoton2@gmail.com>
Date: Sat, 11 May 2024 19:27:05 +0100
Subject: [PATCH 2/2] Re-add useJapanese to configschema but mark as deprecated

---
 configschema.json                   | 7 +++++++
 src/types/schemas/configschema.d.ts | 4 ++++
 2 files changed, 11 insertions(+)

diff --git a/configschema.json b/configschema.json
index 51ffb03b..96ed6851 100644
--- a/configschema.json
+++ b/configschema.json
@@ -193,6 +193,12 @@
           "type": "string",
           "default": "SHORTNAME"
         },
+        "useJapanese": {
+          "$comment": "DEPRECATED; should be removed in next major version (2.x > 3.x).",
+          "description": "DEPRECATED; left for compatibility reasons.",
+          "type": "boolean",
+          "default": false
+        },
         "ignoreGamesWhileImporting": {
           "$ref": "#/definitions/ignoreGamesWhileImporting"
         },
@@ -213,6 +219,7 @@
       },
       "required": [
         "defaultMarathon",
+        "useJapanese",
         "disableSpeedrunComLookup"
       ]
     },
diff --git a/src/types/schemas/configschema.d.ts b/src/types/schemas/configschema.d.ts
index c98a7322..95d485b5 100644
--- a/src/types/schemas/configschema.d.ts
+++ b/src/types/schemas/configschema.d.ts
@@ -37,6 +37,10 @@ export interface Configschema {
 	horaro: Horaro;
 	oengus: {
 		defaultMarathon: string;
+		/**
+		 * DEPRECATED; left for compatibility reasons.
+		 */
+		useJapanese: boolean;
 		ignoreGamesWhileImporting?: IgnoreGamesWhileImporting;
 		disableSpeedrunComLookup: boolean;
 		useSandbox?: boolean | null;