Okay. This is a good explanation, but it would be nice if that was indicated in the UI of the app. It's not obvious that multiple comments are aggregated into a single publish action.
On Tuesday, December 2, 2014 11:58:17 AM UTC-8, Christian Hammond wrote: > > Reviews can cover multiple lines of multiple diffs, of file attachments, > and soon can contain general free-floating comments. To keep an email from > going out every time one of these potentially dozens of comments are made, > we assemble them into a draft, which requires publishing. > > Removing the publishing requirement and enabling some auto-publishing > timer can result in unexpected behavior. There have been many, many times > I've begun a review, been interrupted for a couple hours, came back, > reviewed more, and realized my earlier suggestions in that review were > wrong. If we auto-published, I'd just make things worse for the developer, > possibly causing unnecessary changes to be made. > > So in short, there's no option for this. While it is one extra click, it's > an important one that overall prevents poor experiences. > > Christian > > > On Tuesday, December 2, 2014, Andy Matthews <[email protected] > <javascript:>> wrote: > >> When responding to a review request it's annoying to have to click >> submit, then ALSO click publish. >> >> Is there a setting to allow comments to be automatically published? This >> extra step is needless and presents a poor user experience. >> >> -- >> Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/ >> --- >> Sign up for Review Board hosting at RBCommons: https://rbcommons.com/ >> --- >> Happy user? Let us know at http://www.reviewboard.org/users/ >> --- >> You received this message because you are subscribed to the Google Groups >> "reviewboard" group. >> To unsubscribe from this group and stop receiving emails from it, send an >> email to [email protected]. >> For more options, visit https://groups.google.com/d/optout. >> > > > -- > -- > Christian Hammond - [email protected] <javascript:> > Review Board - http://www.reviewboard.org > Beanbag, Inc. - http://www.beanbaginc.com > > -- Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/ --- Sign up for Review Board hosting at RBCommons: https://rbcommons.com/ --- Happy user? Let us know at http://www.reviewboard.org/users/ --- You received this message because you are subscribed to the Google Groups "reviewboard" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/d/optout.
