It would be nice if the front page of https://codereview.chromium.org/ (both Material and non-Material) reminded us that some of our code reviews might be in Gerrit, and linked straight to the equivalent view.
https://chromium-review.googlesource.com/506373
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/efa2aa744a7f2b7ff3f486c2a79d97dd5aa71348 commit efa2aa744a7f2b7ff3f486c2a79d97dd5aa71348 Author: Aaron Gable <agable@chromium.org> Date: Thu May 18 20:06:22 2017 Add Gerrit links to external Rietveld Bug: 722562 Change-Id: I63e9c7186976f90a8deeacf461d6024e742b8f48 Reviewed-on: https://chromium-review.googlesource.com/506373 Commit-Queue: Aaron Gable <agable@chromium.org> Reviewed-by: Andrew Bonventre <andybons@chromium.org> [modify] https://crrev.com/efa2aa744a7f2b7ff3f486c2a79d97dd5aa71348/appengine/chromium_rietveld/new_static/views/cr-inbox-view.html [modify] https://crrev.com/efa2aa744a7f2b7ff3f486c2a79d97dd5aa71348/appengine/chromium_rietveld/templates/base.html [modify] https://crrev.com/efa2aa744a7f2b7ff3f486c2a79d97dd5aa71348/appengine/chromium_rietveld/templates/user.html [modify] https://crrev.com/efa2aa744a7f2b7ff3f486c2a79d97dd5aa71348/appengine/chromium_rietveld/new_static/components/cr-app.html
The following revision refers to this bug: https://chrome-internal.googlesource.com/infra/infra_internal/+/8c240cdaf8f2510c3ddc4681db46d5ece1d5ce69 commit 8c240cdaf8f2510c3ddc4681db46d5ece1d5ce69 Author: Aaron Gable <agable@chromium.org> Date: Fri May 19 17:18:57 2017
Both internal and external have been deployed.
Lookin' fab – thanks!
Comment 1 by aga...@chromium.org
, May 15 2017Owner: aga...@chromium.org
Status: Started (was: Untriaged)