From f8937165d516cde2c25017d4b05780b9023c3e9a Mon Sep 17 00:00:00 2001
From: Miodec <jack@monkeytype.com>
Date: Fri, 6 Jan 2023 21:40:23 +0100
Subject: [PATCH] showing notification that not all fields are filled closes
 #3888

---
 frontend/src/ts/controllers/account-controller.ts | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/frontend/src/ts/controllers/account-controller.ts b/frontend/src/ts/controllers/account-controller.ts
index 75347d1c6..9b7174e98 100644
--- a/frontend/src/ts/controllers/account-controller.ts
+++ b/frontend/src/ts/controllers/account-controller.ts
@@ -380,6 +380,15 @@ export async function signIn(): Promise<void> {
   const email = ($(".pageLogin .login input")[0] as HTMLInputElement).value;
   const password = ($(".pageLogin .login input")[1] as HTMLInputElement).value;
 
+  if (email === "" || password === "") {
+    Notifications.add("Please fill in all fields", 0);
+    LoginPage.hidePreloader();
+    LoginPage.enableInputs();
+    LoginPage.enableSignUpButton();
+    LoginPage.enableSignInButton();
+    return;
+  }
+
   const persistence = $(".pageLogin .login #rememberMe input").prop("checked")
     ? browserLocalPersistence
     : browserSessionPersistence;