Skip to content

Conversation

zeripath
Copy link
Contributor

Unfortunately fixing changes to mail_issue.go did not get included in #19131.

We also need to not send issue comment mails to deactivated users.

Fix #18950

Signed-off-by: Andrew Thornton art27@cantab.net

Unfortunately fixing changes to `mail_issue.go` did not get included in go-gitea#19131.

We also need to not send issue comment mails to deactivated users.

Fix go-gitea#18950

Signed-off-by: Andrew Thornton <art27@cantab.net>
@zeripath zeripath added type/bug skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. backport/v1.16 labels Mar 19, 2022
@zeripath zeripath added this to the 1.17.0 milestone Mar 19, 2022
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Mar 19, 2022
@zeripath zeripath added the backport/done All backports for this PR have been created label Mar 19, 2022
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Mar 19, 2022
@zeripath zeripath merged commit 3322f4d into go-gitea:main Mar 19, 2022
@zeripath zeripath deleted the dont-send-email-to-inactive-users-part-2 branch March 19, 2022 16:20
@zeripath zeripath changed the title not send notification emails to inactive users (part 2) Do not send notification emails to inactive users (part 2) Mar 19, 2022
zeripath added a commit that referenced this pull request Mar 19, 2022
Backport #19131
Backport #19142

Emails should not be sent to inactive users except for Activate and ResetPassword
messages.

Fix #18950

Signed-off-by: Andrew Thornton <art27@cantab.net>
zjjhot added a commit to zjjhot/gitea that referenced this pull request Mar 21, 2022
* giteaofficial/main:
  Add 1.18 (go-gitea#19151)
  [skip ci] Updated translations via Crowdin
  Fix NPE `/repos/issues/search` when not signed in (go-gitea#19154)
  [skip ci] Updated licenses and gitignores
  Use custom favicon when viewing static files if it exists (go-gitea#19130)
  not send notification emails to inactive users (part 2) (go-gitea#19142)
  Make migrations SKIP_TLS_VERIFY apply to git too (go-gitea#19132)
  Do not send notification emails to inactive users (go-gitea#19131)
Chianina pushed a commit to Chianina/gitea that referenced this pull request Mar 28, 2022
Unfortunately fixing changes to `mail_issue.go` did not get included in go-gitea#19131.

We also need to not send issue comment mails to deactivated users.

Fix go-gitea#18950

Signed-off-by: Andrew Thornton <art27@cantab.net>
@go-gitea go-gitea locked and limited conversation to collaborators Apr 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport/done All backports for this PR have been created lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

mail notifications for releases are send for user with deactivated e-mail
4 participants