Skip to content
This repository has been archived by the owner on Oct 4, 2024. It is now read-only.

Notification Service #197

Merged
merged 6 commits into from
May 29, 2024
Merged

Notification Service #197

merged 6 commits into from
May 29, 2024

Conversation

OG-NI
Copy link
Contributor

@OG-NI OG-NI commented May 27, 2024

closes #134

  • added notificationMethod setting to User (currently only email notifications are supported)
  • notifications can be sent using the notify() method of the NotificationService provided by the NotificationModule
  • email notifications are sent via nodemailer and the variables defined in .env

@OG-NI OG-NI requested a review from henrybrink May 27, 2024 12:56
@OG-NI OG-NI linked an issue May 27, 2024 that may be closed by this pull request
Benachrichtigungsframework entwickeln (mind. E-Mail) #134
Closed
henrybrink
henrybrink approved these changes May 29, 2024
View reviewed changes
Copy link
Collaborator

@henrybrink henrybrink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@henrybrink henrybrink merged commit 7a25a2a into main May 29, 2024
5 checks passed
@OG-NI OG-NI deleted the dev/notification-service branch June 5, 2024 06:35
@henrybrink henrybrink added the backend Issues related to the backend label Jun 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@henrybrink henrybrink henrybrink approved these changes

Assignees

@OG-NI OG-NI

Labels
backend Issues related to the backend
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Benachrichtigungsframework entwickeln (mind. E-Mail)
2 participants
@OG-NI @henrybrink