From eec512c729f0609de9094c70843c9ddd9e374d38 Mon Sep 17 00:00:00 2001 From: Rushil Perera Date: Mon, 23 Sep 2024 17:52:11 -0400 Subject: [PATCH] fix: skip time conversion to be undefined if both are 0 --- .../episodes/getEpisodeUrl/convertSkipTime.ts | 15 +++++++++++---- 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/src/controllers/episodes/getEpisodeUrl/convertSkipTime.ts b/src/controllers/episodes/getEpisodeUrl/convertSkipTime.ts index 5a7d964..53f1060 100644 --- a/src/controllers/episodes/getEpisodeUrl/convertSkipTime.ts +++ b/src/controllers/episodes/getEpisodeUrl/convertSkipTime.ts @@ -4,8 +4,15 @@ export interface SkipTime { } export function convertSkipTime(skipTime: SkipTime): number[] | undefined { - return typeof skipTime?.start === "number" && - typeof skipTime?.end === "number" - ? [skipTime.start, skipTime.end].map((seconds) => Math.floor(seconds)) - : undefined; + if ( + typeof skipTime?.start !== "number" || + typeof skipTime?.end !== "number" + ) { + return undefined; + } + if (skipTime.end === 0) { + return undefined; + } + + return [skipTime.start, skipTime.end].map((seconds) => Math.floor(seconds)); }