Bug 444503 - show Gerrit topic in the dashboard
Summary: show Gerrit topic in the dashboard
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 2.5   Edit
Assignee: Marc-André Laperle CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed, noteworthy
Depends on: 443871
Blocks:
  Show dependency tree
 
Reported: 2014-09-18 12:20 EDT by Marc-André Laperle CLA
Modified: 2014-09-29 14:22 EDT (History)
3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Marc-André Laperle CLA 2014-09-18 12:20:04 EDT 
It would be nice to see the Gerrit topic in the Gerrit Dashboard as a column. This would be a follow up to bug 443871.
Comment 1 Marc-André Laperle CLA 2014-09-23 22:56:23 EDT 
Patch:
https://git.eclipse.org/r/33804
Comment 2 Miles Parker CLA 2014-09-24 15:58:36 EDT 
THe patch looks good, my only concern about this change is that it adds yet another column to the dashboard. And this isn't aligned with the Gerrit Web UI, which at least for me doesn't show topic. (Maybe Ericsson views are configured differently.) If there was a column selector I'd feel really confident about the change.
Comment 3 Miles Parker CLA 2014-09-24 15:58:51 EDT 
Also, Marc-Andre, please assign yourself to bug. ;)
Comment 4 Marc-André Laperle CLA 2014-09-24 16:31:37 EDT 
(In reply to Miles Parker from comment #2)
> THe patch looks good, my only concern about this change is that it adds yet
> another column to the dashboard. And this isn't aligned with the Gerrit Web
> UI, which at least for me doesn't show topic. (Maybe Ericsson views are
> configured differently.) If there was a column selector I'd feel really
> confident about the change.

I thought it was shown as a separate column on the web but it's actually in the same column as the branch with the topic in parenthesis. For example:

master (My Topic)

I can change the patch to do that. How does that sound?
Comment 5 Miles Parker CLA 2014-09-24 16:35:08 EDT 
That sounds much better. Thanks!
Comment 6 Marc-André Laperle CLA 2014-09-29 00:57:50 EDT 
(In reply to Miles Parker from comment #5)
> That sounds much better. Thanks!

I have updated the patch so that the topic is in the same column as the branch. I like it better this way.
Comment 7 Marc-André Laperle CLA 2014-09-29 14:22:59 EDT 
The patch was merged into master.