Skip to content
This repository has been archived by the owner on Aug 13, 2022. It is now read-only.

approval mail not necessary if you have approval rights #146

Closed
rallek opened this issue Jul 12, 2020 · 6 comments
Closed

approval mail not necessary if you have approval rights #146

rallek opened this issue Jul 12, 2020 · 6 comments
Assignees
Milestone

Comments

@rallek
Copy link

rallek commented Jul 12, 2020

If I add a new message to news and approve it directly I do get an email where I am informed that I have approved my message. That is not necessary.

@Guite
Copy link
Member

Guite commented Jul 12, 2020

You are hitting the edge case that the editor and (the only) moderator is the same person.
But there could be other moderators; in that case it would be reasonable to notify them as well.

@Guite
Copy link
Member

Guite commented Jul 12, 2020

For clarification: this is default behaviour of the generator. At the configuration page a user group is selected as moderation group. For each editor change a notification mail is sent to all moderators. For each moderator change a notification is sent to the editor.
If there is anything to be changed that should be done in MOST centrally I guess. But I am a bit unsure whether we should change this because

But there could be other moderators; in that case it would be reasonable to notify them as well.

Opinions? @craigh @gfr

@Guite Guite added this to the 2.1.0 milestone Jul 12, 2020
@craigh
Copy link
Member

craigh commented Jul 12, 2020

I am also unsure.

@Guite Guite self-assigned this Jul 12, 2020
@rallek
Copy link
Author

rallek commented Jul 13, 2020

My proposal:
grafik
If you have the permission to approve no mail will be send. That was the behavior in the old news module I guess.

But I can live with it as it is. I thought it was different in the past (at least for my zk1.5 site it is different)

@Guite
Copy link
Member

Guite commented Jul 14, 2020

Intermediate results after a phone call:

  1. Ralf is going to do some more tests with having in mind how the workflows and notifications are designed (mails between creators and moderators). Also he will carefully check which mails he receives when (mail to creator vs. mail to moderator).

  2. It would be possible to suppress the notification (all notifications) for the creator if this user has approve permissions. This would be a general enhancement in the generator.

@Guite
Copy link
Member

Guite commented Jan 19, 2021

Closing here because this will become available automatically after the feature has been included in the generator.

@Guite Guite closed this as completed Jan 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants