Mail notification on commit #4052

Closed
opened 2025-11-02 05:35:44 -06:00 by GiteaMirror · 13 comments
Owner

Originally created by @dimmuboy on GitHub (Oct 3, 2019).

  • Gitea version (or commit ref): 1.9.3
  • Git version: 2.22.1
  • Operating system: Arch Linux
  • Database (use [x]):
    • PostgreSQL
    • MySQL
    • MSSQL
    • SQLite
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (provide example URL)
    • No
    • Not relevant
  • Log gist:
    None

Description

I don't know how should it works but I don't get mail notifications about new commits.
I have repo in my watchlist it's starred too.
I'm member of team where repo is.
I have correct mail address and mail server setup (test mail works).
In config file is enabled DEFAULT_EMAIL_NOTIFICATIONS and ENABLE_NOTIFY_MAIL

Originally created by @dimmuboy on GitHub (Oct 3, 2019). - Gitea version (or commit ref): 1.9.3 - Git version: 2.22.1 - Operating system: Arch Linux - Database (use `[x]`): - [ ] PostgreSQL - [x] MySQL - [ ] MSSQL - [ ] SQLite - Can you reproduce the bug at https://try.gitea.io: - [ ] Yes (provide example URL) - [ ] No - [x] Not relevant - Log gist: None ## Description I don't know how should it works but I don't get mail notifications about new commits. I have repo in my watchlist it's starred too. I'm member of team where repo is. I have correct mail address and mail server setup (test mail works). In config file is enabled DEFAULT_EMAIL_NOTIFICATIONS and ENABLE_NOTIFY_MAIL
GiteaMirror added the issue/confirmedtype/enhancement labels 2025-11-02 05:35:44 -06:00
Author
Owner

@bobemoe commented on GitHub (Oct 3, 2019):

I don't think this is implemented yet. There is discussion (and possibly a fork with this) along with other notification improvements over at https://github.com/go-gitea/gitea/issues/145

@bobemoe commented on GitHub (Oct 3, 2019): I don't _think_ this is implemented yet. There is discussion (and possibly a fork with this) along with other notification improvements over at https://github.com/go-gitea/gitea/issues/145
Author
Owner

@lunny commented on GitHub (Oct 4, 2019):

We needs a PR to implement that.

@lunny commented on GitHub (Oct 4, 2019): We needs a PR to implement that.
Author
Owner

@stale[bot] commented on GitHub (Dec 3, 2019):

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

@stale[bot] commented on GitHub (Dec 3, 2019): This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.
Author
Owner

@lunny commented on GitHub (Dec 3, 2019):

I remember one of @guillep2k 's PR will fix this.

@lunny commented on GitHub (Dec 3, 2019): I remember one of @guillep2k 's PR will fix this.
Author
Owner

@guillep2k commented on GitHub (Dec 3, 2019):

@lunny Sorry, my PRs were about updating references from commit messages, but commits do not send mail notifications per-se. CommitRepoAction() should handle this, and it creates an action through models.NotifyWatchers, but mails are not generated AFAIK.

@guillep2k commented on GitHub (Dec 3, 2019): @lunny Sorry, my PRs were about updating references from commit messages, but commits do not send mail notifications per-se. `CommitRepoAction()` should handle this, and it creates an `action` through `models.NotifyWatchers`, but mails are not generated AFAIK.
Author
Owner

@lunny commented on GitHub (Dec 3, 2019):

Implement NotifyPushCommits will resolve the issue?

@lunny commented on GitHub (Dec 3, 2019): Implement `NotifyPushCommits` will resolve the issue?
Author
Owner

@guillep2k commented on GitHub (Dec 3, 2019):

It looks like most of CommitRepoAction should be moved to notifications. But for a minimal implementation yes, NotifyPushCommits would be enough.

@guillep2k commented on GitHub (Dec 3, 2019): It looks like most of `CommitRepoAction` should be moved to notifications. But for a minimal implementation yes, `NotifyPushCommits` would be enough.
Author
Owner

@stale[bot] commented on GitHub (Feb 1, 2020):

This issue has been automatically marked as stale because it has not had recent activity. I am here to help clear issues left open even if solved or waiting for more insight. This issue will be closed if no further activity occurs during the next 2 weeks. If the issue is still valid just add a comment to keep it alive. Thank you for your contributions.

@stale[bot] commented on GitHub (Feb 1, 2020): This issue has been automatically marked as stale because it has not had recent activity. I am here to help clear issues left open even if solved or waiting for more insight. This issue will be closed if no further activity occurs during the next 2 weeks. If the issue is still valid just add a comment to keep it alive. Thank you for your contributions.
Author
Owner

@guillep2k commented on GitHub (Feb 1, 2020):

Bump!

@guillep2k commented on GitHub (Feb 1, 2020): Bump!
Author
Owner

@bendem commented on GitHub (Jul 7, 2020):

On top of that, a visual indicator in the PR's conversation about new commit would also help.

See github including commits in the conversation:
image

@bendem commented on GitHub (Jul 7, 2020): On top of that, a visual indicator in the PR's conversation about new commit would also help. See github including commits in the conversation: ![image](https://user-images.githubusercontent.com/2681677/86755021-9bf2a880-c041-11ea-9775-0b5924d52c06.png)
Author
Owner

@alam-R commented on GitHub (Mar 11, 2021):

As a workaround for mailing pushing events, I have created a minimal Java Service to handle a push webhook and map it to mail.
It needs a lot of customization but it can be a workaround for small projects.
For anyone who is interested: https://github.com/alam-R/giteaWebhookMailProxy

Cheers!

@alam-R commented on GitHub (Mar 11, 2021): As a workaround for mailing pushing events, I have created a minimal Java Service to handle a push webhook and map it to mail. It needs a lot of customization but it can be a workaround for small projects. For anyone who is interested: https://github.com/alam-R/giteaWebhookMailProxy Cheers!
Author
Owner

@Zyzto commented on GitHub (Jul 5, 2021):

any news on this ?

@Zyzto commented on GitHub (Jul 5, 2021): any news on this ?
Author
Owner

@techknowlogick commented on GitHub (Jul 8, 2021):

Closing this as emails happen on commits (at least to PRs), and they are added to PR conversations.

@techknowlogick commented on GitHub (Jul 8, 2021): Closing this as emails happen on commits (at least to PRs), and they are added to PR conversations.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#4052