From f50596c4a114c2735403ea09cdf500863d63ad01 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Aleksa=20Siri=C5=A1ki?= <31509435+aleksasiriski@users.noreply.github.com> Date: Wed, 1 Feb 2023 23:11:21 +0100 Subject: [PATCH 1/2] fixed setExpiry() The bug was that setExpiry didn't work if account was enabled and didn't have an expiry already. --- ts/modules/accounts.ts | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/ts/modules/accounts.ts b/ts/modules/accounts.ts index beb3dd5..e13c11d 100644 --- a/ts/modules/accounts.ts +++ b/ts/modules/accounts.ts @@ -1334,9 +1334,7 @@ export class accountsList { const list = this._collectUsers(); let applyList: string[] = []; for (let id of list) { - if (this._users[id].expiry || enableUser) { - applyList.push(id); - } + applyList.push(id); } this._enableExpiryReason.classList.add("unfocused"); let header: string; From 0c102f532450175bdeafa2688b653a3175ce67d9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Aleksa=20Siri=C5=A1ki?= <31509435+aleksasiriski@users.noreply.github.com> Date: Wed, 1 Feb 2023 23:31:55 +0100 Subject: [PATCH 2/2] added checked to admin only because it is confusing if you go straight to `Manually set the username and password.` and you can't go next since nothing is checked above but it becomes hidden --- html/setup.html | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/html/setup.html b/html/setup.html index e228ab7..0e82692 100644 --- a/html/setup.html +++ b/html/setup.html @@ -137,7 +137,7 @@ {{ .lang.Login.authorizeWithJellyfin }}