Consolidate Redundant FindIt Links |
||||
Issue descriptionSometimes FindIt results show duplicates of the same link which is annoying because it looks like there's 5 different links or so when there is actually one. Example: https://screenshot.googleplex.com/67kuCteXpGM All of the (Details) links in this screenshot go to the exact same URL.
,
Aug 25 2016
The styling on multiple links is pretty bad, which probably makes it hard to tell that they are all different. Findit might be returning duplicate results, though.
,
Aug 25 2016
They're not always duplicates but there were cases I saw this morning where there were many duplicates.
,
Aug 25 2016
I'll take this and also change the details link to be the short git hash or something.
,
Aug 29 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/0ecb56ded5c8b66ec5cce3b93b7a5852df150ef0 commit 0ecb56ded5c8b66ec5cce3b93b7a5852df150ef0 Author: Tiffany Zhang <zhangtiff@chromium.org> Date: Mon Aug 29 22:56:53 2016 SoM: De-dupe FindIt results and show short hash. BUG= 641018 Change-Id: Ib4e776bccae7fc70fa9acbb478cc5222d28e534f Reviewed-on: https://chromium-review.googlesource.com/377620 Commit-Queue: Tiffany Zhang <zhangtiff@chromium.org> Reviewed-by: Stephen Martinis <martiniss@chromium.org> [modify] https://crrev.com/0ecb56ded5c8b66ec5cce3b93b7a5852df150ef0/go/src/infra/appengine/sheriff-o-matic/README.md [modify] https://crrev.com/0ecb56ded5c8b66ec5cce3b93b7a5852df150ef0/go/src/infra/appengine/sheriff-o-matic/elements/som-extension-build-failure.html [modify] https://crrev.com/0ecb56ded5c8b66ec5cce3b93b7a5852df150ef0/go/src/infra/appengine/sheriff-o-matic/test/som-extension-build-failure-test.html
,
Aug 30 2016
|
||||
►
Sign in to add a comment |
||||
Comment 1 by seanmccullough@google.com
, Aug 25 2016