Skip to content

Conversation

@partydragen
Copy link
Member

@partydragen partydragen commented May 4, 2025

Adds default notification templates for generic contents can be used for most notifications purposes

Keeping it simply sending notifications

$notification = new Notification(
    'mass_message',
    $title,
    $content,
    ....
)

Default email template always contains link, Perhaps register all notifications to nl2_alerts even if alert preference is off? Then all links can be sent to /user/alerts/?id={ID} then it will direct user + mark as read, or if no link is set in alert it wik simply open the rich content page

Another reason having title, content in notification constructer is for future notification types such as discord, not need to update every single alert line to support a new type

I believe AlertTemplate is not needed thought as it already simply just using the title, content, link etc

@partydragen partydragen changed the title Email template classes example Add notification default templates May 4, 2025
@partydragen partydragen requested a review from tadhgboyle May 4, 2025 13:35
@partydragen partydragen changed the title Add notification default templates Add notification default templates example May 4, 2025
@partydragen partydragen marked this pull request as ready for review May 7, 2025 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants