Skip to content

Notify users on login via e-mail#4824

Open
frjo wants to merge 2 commits intomainfrom
feature/notificate-user-login
Open

Notify users on login via e-mail#4824
frjo wants to merge 2 commits intomainfrom
feature/notificate-user-login

Conversation

@frjo
Copy link
Copy Markdown
Member

@frjo frjo commented Apr 21, 2026

Fixes #3549

Test Steps

  • Check that user receives en e-mail on login.
  • Check that the e-mail content is correct and informative.

@frjo frjo added Type: Feature This is something new (not an enhancement of an existing thing). Type: Patch Mini change, used in release drafter Status: Needs testing Tickets that need testing/qa Status: Needs dev testing 🧑‍💻 Tasks that should be tested by the dev team labels Apr 21, 2026
@wes-otf
Copy link
Copy Markdown
Contributor

wes-otf commented Apr 22, 2026

This also works well & looks nice - is there a way to skip this when hijack/"switch user" is utilized? not a huge deal but would be nice to not send a notification for that

@frjo
Copy link
Copy Markdown
Member Author

frjo commented Apr 22, 2026

This also works well & looks nice - is there a way to skip this when hijack/"switch user" is utilized?

This I missed! No e-mail should be send in these cases I think, will take a look at that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status: Needs dev testing 🧑‍💻 Tasks that should be tested by the dev team Status: Needs testing Tickets that need testing/qa Type: Feature This is something new (not an enhancement of an existing thing). Type: Patch Mini change, used in release drafter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make Hypha send an e-mail to the user on each successful login

2 participants