New issue
Advanced search Search tips

Issue 605984 link

Starred by 3 users

Issue metadata

Status: WontFix
Owner: ----
Closed: Aug 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocked on:
issue 600469



Sign in to add a comment

codereview - email responses and LGTMs should capture the sender's username, not 'chromium-reviews'

Project Member Reported by benhenry@google.com, Apr 22 2016

Issue description

From email thread:
"""
On this CL:

https://codereview.chromium.org/1907093002/

We see a message in codereview that is from "chromium-reviews" with content "lgtm". Actually, the original author of that is vogelheim@ and on the email thread (rather than codereview UI) I can actually see him as the author.

However, in codereview UI there's no indication of who's the author.

I've also observed this on some past code reviews as well when I was doing some code archealogoy where the content of the message was more meaty and it was really confusing to not see an actual person's name next to the message.

Is this a known issue?
"""
 

Comment 1 by aga...@chromium.org, Apr 27 2016

Components: Infra>Codereview
Labels: -Infra-Codereview
Project Member

Comment 2 by sheriffbot@chromium.org, Jun 2 2016

Labels: Hotlist-Google
Blockedon: 600469
Status: Available (was: Untriaged)
This looks like it’s because the user replied via email.
Cc: wolenetz@chromium.org
 Issue 455950  has been merged into this issue.
Project Member

Comment 5 by sheriffbot@chromium.org, Aug 14 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 6 by aga...@chromium.org, Aug 16 2017

Components: -Infra>Codereview Infra>Codereview>Rietveld
Status: WontFix (was: Untriaged)
Rietveld is obsoleted by gerrit.

Sign in to add a comment