--
--
To unsubscribe, email repo-discuss...@googlegroups.com
More info at http://groups.google.com/group/repo-discuss?hl=en
---
You received this message because you are subscribed to the Google Groups "Repo and Gerrit Discussion" group.
To unsubscribe from this group and stop receiving emails from it, send an email to repo-discuss...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
> However, we may want to consider just modifying the existing notify types to not receive notifications on "fresh" WIP changes.Yes, please. Someone uploading a fresh WIP change is a private event. I noticed recently that when such a change is abandoned without starting review, I get notifications about it. The change you're proposing would make it so that I don't have to notice such a change at all.What I'm starting to wonder about is whether there should be a watch type <https://gerrit-review.googlesource.com/Documentation/rest-api-accounts.html#project-watch-info> for people who want to see all events, including events for fresh WIP changes (and including changes made with notify=NONE). I would expect that to take care of those unusual use cases where some system really wants an email for every event.
Thoughts?Jonathan
Le mer. 24 mai 2017 à 14:44, 'Logan Hanks' via Repo and Gerrit Discussion <repo-d...@googlegroups.com> a écrit :
I'm working on applying WIP to notifications (code in progress). One requirement is to have a way for a watcher to observe everything that happens only on changes that have reached a point of starting review. I was thinking of adding a new notify type, called "review_started_changes", to provide that.--However, we may want to consider just modifying the existing notify types to not receive notifications on "fresh" WIP changes. That is, if a change is created in the WIP state, we could just default the notify level below ALL (I'm thinking OWNER) on all notifications. This is, in fact, what I've proposed all along, but I want to make sure this doesn't surprise anyone depending on watching new_changes, new_patchsets, etc.The upside:1. A user can upload a change, never start review, and never trigger an email about it.2. We have the option to avoid adding a new, potentially confusing notify type.The downside:1. Watchers no longer have a guarantee that they are seeing *every* change. That may be ok for a project, because WIP changes can't be merged without starting review and triggering notifications. But, eager watchers would miss out on early notification that a change is coming, and a malicious user may be able to upload content without immediate observation.Would anyone be unpleasantly surprised by missing out on notifications of never-reviewed WIP changes? If not, I think that may provide a better user experience.
--
To unsubscribe, email repo-discuss+unsubscribe@googlegroups.com
More info at http://groups.google.com/group/repo-discuss?hl=en
---
You received this message because you are subscribed to the Google Groups "Repo and Gerrit Discussion" group.
To unsubscribe from this group and stop receiving emails from it, send an email to repo-discuss+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
More info at http://groups.google.com/group/repo-discuss?hl=en
---
You received this message because you are subscribed to the Google Groups "Repo and Gerrit Discussion" group.
To unsubscribe from this group and stop receiving emails from it, send an email to repo-discuss+unsubscribe@googlegroups.com.
I think there might be some confusion. When Logan says "notifications" he means "emails", which are (AFAIK) the only thing controlled by the notify preferences and REST API arguments.It sounds like Matthias is talking about the *Listener extension points and/or stream-events, which is unaffected by Logan's proposal.