[PR #457] [MERGED] feat: add ability to send login code via email #786

New Issue

OVERLORD · 2025-10-07T00:22:21+03:00

OVERLORD commented

2025-10-07 00:22:21 +03:00

📋 Pull Request Information

Original PR: https://github.com/pocket-id/pocket-id/pull/457
Author: @stonith404
Created: 4/18/2025
Status: ✅ Merged
Merged: 4/20/2025
Merged by: @stonith404

Base: main ← Head: feat/login-code-email

📝 Commits (9)

1c117bd feat: add ability to send login code via email
36f636d Merge branch 'main' into feat/login-code-email
309aab7 use switch statement
abd25f8 update tests to use for button text
ac79cf6 Merge branch 'main' into feat/login-code-email
486ab5c Merge branch 'main' into feat/login-code-email
13b2703 add ability to disable or enable login code via email
0dc58ea Merge remote-tracking branch 'origin/feat/login-code-email' into feat/login-code-email
b9a89a1 fix e2e tests

📊 Changes

22 files changed (+256 additions, -136 deletions)

View changed files

📝 backend/internal/controller/user_controller.go (+50 -4)
📝 backend/internal/dto/app_config_dto.go (+34 -33)
📝 backend/internal/dto/user_dto.go (+5 -1)
📝 backend/internal/model/app_config.go (+14 -11)
📝 backend/internal/service/app_config_service.go (+2 -6)
📝 backend/internal/service/app_config_service_test.go (+0 -38)
📝 backend/internal/service/email_service.go (+3 -3)
📝 backend/internal/service/email_service_templates.go (+1 -0)
📝 backend/internal/service/user_service.go (+30 -13)
➕ backend/internal/utils/date_time_util.go (+52 -0)
📝 backend/resources/email-templates/one-time-access_html.tmpl (+1 -1)
📝 backend/resources/email-templates/one-time-access_text.tmpl (+1 -1)
📝 frontend/messages/en-US.json (+8 -2)
📝 frontend/src/lib/components/one-time-link-modal.svelte (+28 -8)
📝 frontend/src/lib/services/user-service.ts (+5 -1)
📝 frontend/src/lib/types/application-configuration.ts (+2 -1)
📝 frontend/src/routes/login/alternative/+page.svelte (+1 -1)
📝 frontend/src/routes/settings/admin/application-configuration/+page.svelte (+1 -1)
📝 frontend/src/routes/settings/admin/application-configuration/forms/app-config-email-form.svelte (+12 -7)
📝 frontend/src/routes/settings/admin/users/user-list.svelte (+1 -1)

...and 2 more files

📄 Description

Closes #435

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/pocket-id/pocket-id/pull/457 **Author:** [@stonith404](https://github.com/stonith404) **Created:** 4/18/2025 **Status:** ✅ Merged **Merged:** 4/20/2025 **Merged by:** [@stonith404](https://github.com/stonith404) **Base:** `main` ← **Head:** `feat/login-code-email` --- ### 📝 Commits (9) - [`1c117bd`](https://github.com/pocket-id/pocket-id/commit/1c117bd457b55ede306176ef67cd4991e4fd0df4) feat: add ability to send login code via email - [`36f636d`](https://github.com/pocket-id/pocket-id/commit/36f636dcd09971793b230d8cd3aba33f4b00dd18) Merge branch 'main' into feat/login-code-email - [`309aab7`](https://github.com/pocket-id/pocket-id/commit/309aab75b14a5d0b2e6b69e52a79ab2d2cb88c37) use switch statement - [`abd25f8`](https://github.com/pocket-id/pocket-id/commit/abd25f854881bec72dcca36be1e0ddc0c1dbff9c) update tests to use for button text - [`ac79cf6`](https://github.com/pocket-id/pocket-id/commit/ac79cf6d0181876935cad55f7af6e898c64c0e31) Merge branch 'main' into feat/login-code-email - [`486ab5c`](https://github.com/pocket-id/pocket-id/commit/486ab5c2a109c847b69a420c4fd805c0a2702d65) Merge branch 'main' into feat/login-code-email - [`13b2703`](https://github.com/pocket-id/pocket-id/commit/13b27033053d985fad8c6a465def1f5ac5c2dac4) add ability to disable or enable login code via email - [`0dc58ea`](https://github.com/pocket-id/pocket-id/commit/0dc58eaa5157c44f546c975aa5687553743f802f) Merge remote-tracking branch 'origin/feat/login-code-email' into feat/login-code-email - [`b9a89a1`](https://github.com/pocket-id/pocket-id/commit/b9a89a1ad896145028f7cbce258355e090975c14) fix e2e tests ### 📊 Changes **22 files changed** (+256 additions, -136 deletions) <details> <summary>View changed files</summary> 📝 `backend/internal/controller/user_controller.go` (+50 -4) 📝 `backend/internal/dto/app_config_dto.go` (+34 -33) 📝 `backend/internal/dto/user_dto.go` (+5 -1) 📝 `backend/internal/model/app_config.go` (+14 -11) 📝 `backend/internal/service/app_config_service.go` (+2 -6) 📝 `backend/internal/service/app_config_service_test.go` (+0 -38) 📝 `backend/internal/service/email_service.go` (+3 -3) 📝 `backend/internal/service/email_service_templates.go` (+1 -0) 📝 `backend/internal/service/user_service.go` (+30 -13) ➕ `backend/internal/utils/date_time_util.go` (+52 -0) 📝 `backend/resources/email-templates/one-time-access_html.tmpl` (+1 -1) 📝 `backend/resources/email-templates/one-time-access_text.tmpl` (+1 -1) 📝 `frontend/messages/en-US.json` (+8 -2) 📝 `frontend/src/lib/components/one-time-link-modal.svelte` (+28 -8) 📝 `frontend/src/lib/services/user-service.ts` (+5 -1) 📝 `frontend/src/lib/types/application-configuration.ts` (+2 -1) 📝 `frontend/src/routes/login/alternative/+page.svelte` (+1 -1) 📝 `frontend/src/routes/settings/admin/application-configuration/+page.svelte` (+1 -1) 📝 `frontend/src/routes/settings/admin/application-configuration/forms/app-config-email-form.svelte` (+12 -7) 📝 `frontend/src/routes/settings/admin/users/user-list.svelte` (+1 -1) _...and 2 more files_ </details> ### 📄 Description Closes #435 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

OVERLORD added the pull-request label 2025-10-07 00:22:21 +03:00

OVERLORD closed this issue

2025-10-07 00:22:22 +03:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/pocket-id#786