Issue 2229: Gerrit review update should also return the URL
Status:  Released
Owner: ----
Closed:  Nov 2013
Reported by Lars.Vo...@gmail.com, Oct 31, 2013
As discussed with Shawn on the EclipseCon Europe it would be helpful IMHO if Gerrit returns also the URL for an updated Gerrit review.

Currently Gerrit returns the URL only during the initial creation. This makes finding the updated Gerrit review a little bit more complex than necessary.
Oct 31, 2013
Project Member #1 edwin.ke...@gmail.com
What exactly do you mean by 'updated Gerrit review'? Can you please say which SSH command or REST API you talking about?
Oct 31, 2013
#2 Lars.Vo...@gmail.com
I push via Eclipse. First push returns the following:

Repository ssh://lvogel@git.eclipse.org:29418/platform/eclipse.platform.ui.git

Processing changes: new: 1, refs: 1
Processing changes: new: 1, refs: 1, done    
----------
Reviewing commit: commit 0b0c70e59dbeb2a8905978e246ceaaebefe05206 1383224863 ----sp
Authored by: Lars Vogel <Lars.Vogel@gmail.com>

The author is a committer on the project.

This commit passes Eclipse validation.

New Changes:
  https://git.eclipse.org/r/17921


The update returns the following:

Repository ssh://lvogel@git.eclipse.org:29418/platform/eclipse.platform.ui.git

Processing changes: updated: 1, refs: 1
Processing changes: updated: 1, refs: 1, done    
----------
Reviewing commit: commit 87d89f3cb08c0f6115832b0ee52e18a2ba192758 1383224898 ----sp
Authored by: Lars Vogel <Lars.Vogel@gmail.com>

The author is a committer on the project.

This commit passes Eclipse validation.
(W) 87d89f3: no files changed, message updated

-----------

Would be great if the second answer also include in this example the URL:

https://git.eclipse.org/r/17921
Oct 31, 2013
Project Member #3 edwin.ke...@gmail.com
(No comment was entered for this change.)
Summary: Gerrit review update should also return the URL (was: Gerrit review update should also return the ULR)
Oct 31, 2013
Project Member #4 edwin.ke...@gmail.com
https://gerrit-review.googlesource.com/51285
Status: ChangeUnderReview
Oct 31, 2013
#5 Lars.Vo...@gmail.com
Thanks Edwin, via Twitter the Eclipse community cheered with happiness after your uploaded review. :-)
Nov 5, 2013
Project Member #6 edwin.ke...@gmail.com
(No comment was entered for this change.)
Status: Submitted
Labels: FixedIn-2.9
May 20, 2014
Project Member #7 edwin.ke...@gmail.com
 Issue 2671  has been merged into this issue.
Jul 18, 2014
Project Member #8 edwin.ke...@gmail.com
(No comment was entered for this change.)
Status: Released