From 22593ab435eae99b72dfaed5cf8aa6b7ae2a2c87 Mon Sep 17 00:00:00 2001 From: Sergio Vera Date: Sun, 12 May 2024 20:32:09 +0200 Subject: [PATCH 1/3] Fixed spinner in reader dark mode --- internal/webserver/embedded/css/reader.css | 4 ++++ internal/webserver/embedded/views/reader.html | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/internal/webserver/embedded/css/reader.css b/internal/webserver/embedded/css/reader.css index 39803a0..ebabbcf 100644 --- a/internal/webserver/embedded/css/reader.css +++ b/internal/webserver/embedded/css/reader.css @@ -231,3 +231,7 @@ body { fill: Canvas; filter: drop-shadow(0 1px 0 rgba(0, 0, 0, .2)); } +.spinner { + fill: #888; + stroke: #888; +} diff --git a/internal/webserver/embedded/views/reader.html b/internal/webserver/embedded/views/reader.html index 011db87..4d1f8b4 100644 --- a/internal/webserver/embedded/views/reader.html +++ b/internal/webserver/embedded/views/reader.html @@ -17,7 +17,7 @@
- +
From fb2bac175c9146acf91e316d09e5e31994279a15 Mon Sep 17 00:00:00 2001 From: Sergio Vera Date: Sun, 12 May 2024 20:33:15 +0200 Subject: [PATCH 2/3] Fixed send to email not address not being saved when creating a new user --- internal/webserver/controller/user/create.go | 13 +++---- internal/webserver/user_management_test.go | 37 ++++++++++---------- 2 files changed, 26 insertions(+), 24 deletions(-) diff --git a/internal/webserver/controller/user/create.go b/internal/webserver/controller/user/create.go index ece86b1..c31eb68 100644 --- a/internal/webserver/controller/user/create.go +++ b/internal/webserver/controller/user/create.go @@ -14,12 +14,13 @@ import ( func (u *Controller) Create(c *fiber.Ctx) error { role, _ := strconv.Atoi(c.FormValue("role")) user := model.User{ - Name: c.FormValue("name"), - Username: strings.ToLower(c.FormValue("username")), - Email: c.FormValue("email"), - Password: c.FormValue("password"), - Role: role, - Uuid: uuid.NewString(), + Name: c.FormValue("name"), + Username: strings.ToLower(c.FormValue("username")), + Email: c.FormValue("email"), + SendToEmail: c.FormValue("send-to-email"), + Password: c.FormValue("password"), + Role: role, + Uuid: uuid.NewString(), } user.WordsPerMinute, _ = strconv.ParseFloat(c.FormValue("words-per-minute"), 64) diff --git a/internal/webserver/user_management_test.go b/internal/webserver/user_management_test.go index 203e359..598337a 100644 --- a/internal/webserver/user_management_test.go +++ b/internal/webserver/user_management_test.go @@ -29,6 +29,8 @@ func TestUserManagement(t *testing.T) { ) reset := func() { + t.Helper() + var err error db = infrastructure.Connect("file::memory:", 250) app = bootstrapApp(db, &infrastructure.NoEmail{}, afero.NewMemMapFs(), webserver.Config{}) @@ -65,10 +67,8 @@ func TestUserManagement(t *testing.T) { } } - reset() - t.Run("Try to add a user without an active session", func(t *testing.T) { - t.Cleanup(reset) + reset() response, err := getRequest(&http.Cookie{}, app, "/en/users/new", t) if response == nil { @@ -96,7 +96,7 @@ func TestUserManagement(t *testing.T) { }) t.Run("Try to add a user with an admin active session", func(t *testing.T) { - t.Cleanup(reset) + reset() response, err := getRequest(adminCookie, app, "/en/users/new", t) if response == nil { @@ -109,6 +109,7 @@ func TestUserManagement(t *testing.T) { "name": {"New user"}, "username": {"new"}, "email": {"new@example.com"}, + "send-to-email": {"send@example.com"}, "password": {"new"}, "confirm-password": {"new"}, "role": {fmt.Sprint(model.RoleRegular)}, @@ -130,7 +131,7 @@ func TestUserManagement(t *testing.T) { }) t.Run("Try to add a user with a regular user active session", func(t *testing.T) { - t.Cleanup(reset) + reset() response, err := getRequest(regularUserCookie, app, "/en/users/new", t) if response == nil { @@ -148,7 +149,7 @@ func TestUserManagement(t *testing.T) { }) t.Run("Try to add a user with errors in form", func(t *testing.T) { - t.Cleanup(reset) + reset() response, err := postRequest(url.Values{}, adminCookie, app, "/en/users/new", t) expectedErrorMessages := []string{ @@ -167,7 +168,7 @@ func TestUserManagement(t *testing.T) { }) t.Run("Try to add a user with already registered email and username", func(t *testing.T) { - t.Cleanup(reset) + reset() newUserData := url.Values{ "name": {"Test user"}, @@ -192,7 +193,7 @@ func TestUserManagement(t *testing.T) { }) t.Run("Try to update a user without an active session", func(t *testing.T) { - t.Cleanup(reset) + reset() response, err := getRequest(&http.Cookie{}, app, fmt.Sprintf("/en/users/%s/edit", regularUser.Username), t) if response == nil { @@ -210,7 +211,7 @@ func TestUserManagement(t *testing.T) { }) t.Run("Try to update a user using another, non admin user session", func(t *testing.T) { - t.Cleanup(reset) + reset() adminUserData := regularUserData adminUserData.Set("id", adminUser.Uuid) @@ -231,7 +232,7 @@ func TestUserManagement(t *testing.T) { }) t.Run("Try to update the user in session", func(t *testing.T) { - t.Cleanup(reset) + reset() regularUserData.Set("name", "Updated regular user") regularUserData.Set("id", regularUser.Uuid) @@ -255,7 +256,7 @@ func TestUserManagement(t *testing.T) { }) t.Run("Try to update a user with an admin session", func(t *testing.T) { - t.Cleanup(reset) + reset() regularUserData.Set("name", "Updated regular user by an admin") regularUserData.Set("id", regularUser.Uuid) @@ -279,7 +280,7 @@ func TestUserManagement(t *testing.T) { }) t.Run("Try to edit a non existing user with an admin session", func(t *testing.T) { - t.Cleanup(reset) + reset() response, err := getRequest(adminCookie, app, fmt.Sprintf("/en/users/%s/edit", "abcde"), t) if response == nil { @@ -289,7 +290,7 @@ func TestUserManagement(t *testing.T) { }) t.Run("Try to update a non existing user with an admin session", func(t *testing.T) { - t.Cleanup(reset) + reset() regularUserData.Set("name", "Updated test user by an admin") @@ -301,7 +302,7 @@ func TestUserManagement(t *testing.T) { }) t.Run("Try to delete a user without an active session", func(t *testing.T) { - t.Cleanup(reset) + reset() regularUserData = url.Values{ "id": {regularUser.Uuid}, @@ -316,7 +317,7 @@ func TestUserManagement(t *testing.T) { }) t.Run("Try to delete a user with a regular user's session", func(t *testing.T) { - t.Cleanup(reset) + reset() regularUserData = url.Values{ "id": {regularUser.Uuid}, @@ -333,7 +334,7 @@ func TestUserManagement(t *testing.T) { }) t.Run("Try to delete a user with an admin session", func(t *testing.T) { - t.Cleanup(reset) + reset() regularUserData = url.Values{ "id": {regularUser.Uuid}, @@ -352,7 +353,7 @@ func TestUserManagement(t *testing.T) { }) t.Run("Try to delete the only existing admin user", func(t *testing.T) { - t.Cleanup(reset) + reset() regularUserData = url.Values{ "id": {adminUser.Uuid}, @@ -366,7 +367,7 @@ func TestUserManagement(t *testing.T) { }) t.Run("Try to delete a non existing user with an admin session", func(t *testing.T) { - t.Cleanup(reset) + reset() regularUserData = url.Values{ "id": {"abcde"}, From 299a1e79a7d85406dc272bf5ed51cb51d78c1d2c Mon Sep 17 00:00:00 2001 From: Sergio Vera Date: Sun, 12 May 2024 20:41:33 +0200 Subject: [PATCH 3/3] Added missing translations --- internal/webserver/embedded/translations/es.yml | 1 + internal/webserver/embedded/translations/fr.yml | 1 + 2 files changed, 2 insertions(+) diff --git a/internal/webserver/embedded/translations/es.yml b/internal/webserver/embedded/translations/es.yml index 43d704c..fb5783a 100644 --- a/internal/webserver/embedded/translations/es.yml +++ b/internal/webserver/embedded/translations/es.yml @@ -46,6 +46,7 @@ "Edit user": "Editar usuario" "Name cannot be empty": "Debes introducir un nombre" "Incorrect email address": "Dirección de correo electrónico incorrecta" +"Incorrect send to email address": "Dirección de correo electrónico de envío incorrecta" "Incorrect reading speed": "Velocidad de lectura incorrecta" "A user with this email address already exist": "Ya existe un usuario con esta dirección de correo electrónico" "Incorrect role": "Rol incorrecto" diff --git a/internal/webserver/embedded/translations/fr.yml b/internal/webserver/embedded/translations/fr.yml index d17c1f3..8bcd1c0 100644 --- a/internal/webserver/embedded/translations/fr.yml +++ b/internal/webserver/embedded/translations/fr.yml @@ -46,6 +46,7 @@ "Edit user": "Modifier l'utilisateur" "Name cannot be empty": "Le nom ne peut pas être vide" "Incorrect email address": "Adresse e-mail incorrecte" +"Incorrect send to email address": "Adresse e-mail d'envoi incorrecte" "Incorrect reading speed": "Vitesse de lecture incorrecte" "A user with this email address already exist": "Un utilisateur avec cette adresse e-mail existe déjà" "Incorrect role": "Rôle incorrect"