Skip to content

feat: Add email notifications setup#142

Merged
cengique merged 5 commits into
developfrom
feat/notifs
May 15, 2026
Merged

feat: Add email notifications setup#142
cengique merged 5 commits into
developfrom
feat/notifs

Conversation

@KobenjiSan
Copy link
Copy Markdown
Collaborator

  • Added email notifications for comment replies to comment owners (sends only if preferences allow)

  • Redesigned profile settings page

  • Fixed profile image display issues

  • Added Notification Preferences section

    • Included form to update comment notification preferences
  • Added ReceiveCommentReplyEmails boolean to User table to store preferences

- Added email notifications for comment replies to comment owners (sends only if preferences allow)

- Redesigned profile settings page
- Fixed profile image display issues
- Added Notification Preferences section
    - Included form to update comment notification preferences

- Added ReceiveCommentReplyEmails boolean to User table to store preferences
Copy link
Copy Markdown
Member

@cengique cengique left a comment

Choose a reason for hiding this comment

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

I received the emails! but please fix the conflicts.

@KobenjiSan KobenjiSan requested a review from cengique May 12, 2026 22:42
@KobenjiSan
Copy link
Copy Markdown
Collaborator Author

appsettings.development.json will need to be updated for email notifications to continue working. README.md was updated with this required change.

Copy link
Copy Markdown
Member

@cengique cengique left a comment

Choose a reason for hiding this comment

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

works

@cengique cengique merged commit 4626bbf into develop May 15, 2026
2 checks passed
@cengique cengique deleted the feat/notifs branch May 15, 2026 23:25
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