Last modified: 2012-05-28 17:56:40 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T28293, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 26293 - Very bad email grouping/threading for CodeReview mails in Gmail
Very bad email grouping/threading for CodeReview mails in Gmail
Status: NEW
Product: MediaWiki extensions
Classification: Unclassified
CodeReview (Other open bugs)
unspecified
All All
: Low minor (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-12-10 00:02 UTC by p858snake
Modified: 2012-05-28 17:56 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description p858snake 2010-12-10 00:02:03 UTC
Just sticking this here before we forget about it:

> <p858snake> meh thats not as much as a pain as the whole grouping thing since the subject style change in cr
> <Reedy> p858snake, RoanKattouw_away wanted to do the change to stop it grouping for him...
> <Reedy> (and presumably others)
> <p858snake> stop it grouping? it just started
> <Reedy> Roan had it grouping on [Mediawiki]
> <p858snake> i didn't, although admittedly i didn't comment all that much
> <Reedy> hmm
> <Reedy> hopefully
> <Reedy> it should group on [MediaWiki r11111]
> <Reedy> and keep all revision stuff on one
> <p858snake> its grouped at least two differnt code tickets to gether under the one subject now
> * Reedy shrugs
> <Reedy> I don't use gmail for that
> <RoanKattouw> Reedy: It's *still* grouping stuff inappropriately :(
> <Reedy> Boooooo
> <p858snake> i wonder if we could fake the mailing list stuff for that to give them individual ids and group based on that since appropriately that is how mailman does it
> <RoanKattouw> Then we should have IDs per rev
> <RoanKattouw> So all comments on the same rev end up in the same thread
> <RoanKattouw> That's actually a quite good idea
> <p858snake> s/apparently/appropriately/
> <p858snake> http://localhostr.com/files/vvz3gBJ/CR.png
> <p858snake> although its strange that it was grouping them for some people before and not others
> <Platonides> p858snake, there was some revision before a note about adding references to CR emails
> <Platonides> I'd like having a fake from header, too
> <p858snake> we should probably document this somewhere like in bz before we forget about it
Comment 1 Sam Reed (reedy) 2011-03-05 21:04:49 UTC
Is this still an issue?
Comment 2 Roan Kattouw 2011-03-05 22:14:47 UTC
Yes :(

I'd like to experiment with not using any brackets at all in the subject line. I fear Gmail may be confusing them for mailing list prefixes (currently, subject lines look like "[mediawiki-codereview] [MediaWiki r12345] Blah"), or maybe postfixing the revid instead of prefixing it.
Comment 3 Brion Vibber 2011-06-07 22:34:52 UTC
Updated summary to clarify that CodeReview's emails are being horribly, *horribly* misgrouped/misthreaded at least in Gmail.

Gmail will group a bunch of things with this name pattern together as one conversation even though they all have different revision numbers in the subject:
'[MediaWiki r12345]: Follow-up changes'

Likewise, you'll see another thread like:
'[MediaWiki r12345]: Revision status changed'

It's kinda suck! Especially now that we've moved staff to Google Apps / Gmail, we probably have most of our code reviewers reading their mails on this system and it's playing hell with things, especially when a lot of new mails come through during a code review sprint.

Bumping up the priority on this.
Comment 4 Roan Kattouw 2011-06-08 11:23:01 UTC
(In reply to comment #3)
> Updated summary to clarify that CodeReview's emails are being horribly,
> *horribly* misgrouped/misthreaded at least in Gmail.
> 
> Gmail will group a bunch of things with this name pattern together as one
> conversation even though they all have different revision numbers in the
> subject:
> '[MediaWiki r12345]: Follow-up changes'
> 
> Likewise, you'll see another thread like:
> '[MediaWiki r12345]: Revision status changed'
> 
> It's kinda suck! Especially now that we've moved staff to Google Apps / Gmail,
> we probably have most of our code reviewers reading their mails on this system
> and it's playing hell with things, especially when a lot of new mails come
> through during a code review sprint.
> 
FINALLY someone feels my pain :D

Yeah basically what Gmail does now is group all 'Revision status changed' messages together (this is actually good because I don't care about those), all 'Follow-up changes' messages together, all 'New comment added' messages together and all 'New comment added and revision status changed' messages together. This happens until there are 61 messages in the group, after which it moves on to a new group. This is kind of terrible for reading because you're forced to read things in bulk.
Comment 5 Casey Brown 2011-06-08 20:19:01 UTC
Gmail sucks.  Sometimes its threading works amazingly, sometimes it doesn't.  I'm not really sure if there's anyway for us to fix this ourselves.

FWIF, the same issue happens with Bugzilla e-mails where the "New bug" e-mail sorts by itself and then all follow-up mails get sorted together separately, until the bug is renamed and it starts another group.  It would be good if we could somehow tell Gmail that everything with the same bug number or the same revision should be sorted together, but I'm not sure if that's possible.
Comment 6 Roan Kattouw 2011-06-08 20:24:36 UTC
(In reply to comment #5)
> Gmail sucks.  Sometimes its threading works amazingly, sometimes it doesn't. 
> I'm not really sure if there's anyway for us to fix this ourselves.
> 
> FWIF, the same issue happens with Bugzilla e-mails where the "New bug" e-mail
> sorts by itself and then all follow-up mails get sorted together separately,
> until the bug is renamed and it starts another group.
That's reasonable though, because the messages it's grouping have identical subject lines. This is not the case for the CR e-mails we're sending, where the subject lines are subtly different (revision IDs). Yet Gmail groups them anyway, presumably because of the [brackets]. We can fix this by organizing the subject lines differently, presumably.

> It would be good if we
> could somehow tell Gmail that everything with the same bug number or the same
> revision should be sorted together, but I'm not sure if that's possible.
There is an X-Reply-To header (or whatever it's called) that allows you to specify the ID of the message that this is a reply to (this requires you set IDs on messages as well, of course). Ashar wanted to experiment with this.
Comment 7 Chad H. 2011-06-08 20:48:51 UTC
(In reply to comment #6)
> There is an X-Reply-To header (or whatever it's called) that allows you to
> specify the ID of the message that this is a reply to (this requires you set
> IDs on messages as well, of course). Ashar wanted to experiment with this.

Yes. Supposedly as a prerequisite he had to enable the "sending an e-mail to yourself when you do something" process, which nobody liked and we backed out :)
Comment 8 Mark A. Hershberger 2012-05-28 17:56:40 UTC
Lowering priority on high priority bugs that have a low severity

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links