[PR #5110] [MERGED] Refactor mail notification #17644

Closed
opened 2025-11-02 14:41:18 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/go-gitea/gitea/pull/5110
Author: @lunny
Created: 10/19/2018
Status: Merged
Merged: 1/13/2019
Merged by: @lafriks

Base: masterHead: lunny/notification_mail


📝 Commits (3)

  • 4ab7928 mail notification implement interface
  • 9d13dc6 fix file comment year
  • b3f3a13 use NullNotifier as parent struct of notifiers

📊 Changes

8 files changed (+190 additions, -55 deletions)

View changed files

📝 models/issue.go (+0 -3)
📝 models/issue_comment.go (+5 -4)
📝 models/pull.go (+0 -2)
📝 modules/notification/base/notifier.go (+0 -0)
modules/notification/base/null.go (+108 -0)
modules/notification/mail/mail.go (+74 -0)
📝 modules/notification/notification.go (+2 -0)
📝 modules/notification/ui/ui.go (+1 -46)

📄 Description

extract from #4001


🔄 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/go-gitea/gitea/pull/5110 **Author:** [@lunny](https://github.com/lunny) **Created:** 10/19/2018 **Status:** ✅ Merged **Merged:** 1/13/2019 **Merged by:** [@lafriks](https://github.com/lafriks) **Base:** `master` ← **Head:** `lunny/notification_mail` --- ### 📝 Commits (3) - [`4ab7928`](https://github.com/go-gitea/gitea/commit/4ab79283d32c6ccedd6de25e1a4e6a52b8a096df) mail notification implement interface - [`9d13dc6`](https://github.com/go-gitea/gitea/commit/9d13dc69a45a95431500ba50b8dc4f24b388090d) fix file comment year - [`b3f3a13`](https://github.com/go-gitea/gitea/commit/b3f3a13e7be4bbbb0963de2c5b62dae5019cbdc4) use NullNotifier as parent struct of notifiers ### 📊 Changes **8 files changed** (+190 additions, -55 deletions) <details> <summary>View changed files</summary> 📝 `models/issue.go` (+0 -3) 📝 `models/issue_comment.go` (+5 -4) 📝 `models/pull.go` (+0 -2) 📝 `modules/notification/base/notifier.go` (+0 -0) ➕ `modules/notification/base/null.go` (+108 -0) ➕ `modules/notification/mail/mail.go` (+74 -0) 📝 `modules/notification/notification.go` (+2 -0) 📝 `modules/notification/ui/ui.go` (+1 -46) </details> ### 📄 Description extract from #4001 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2025-11-02 14:41:18 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#17644