Closed Bug 1143132 Opened 6 years ago Closed 6 years ago

emails about redirected attachment urls link to attachment-edit instead of the actual url (mozreview, github, etc)

Categories

(bugzilla.mozilla.org :: Extensions, defect)

Production
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: gps, Assigned: glob)

Details

Attachments

(1 file)

HTML emails for review requests from MozReview look something like the following:

---

Ed Morley [:edmorley] <emorley@mozilla.com> has asked Gregory Szorc [:gps]
<gps@mozilla.com> for review:
Bug 980986: qimportbz incorrectly prompts to overwrite patch if filename on bug
was equal to the bug number
https://bugzilla.mozilla.org/show_bug.cgi?id=980986

Attachment 8577204 [details]: MozReview Request: bz://980986/edmorley
https://bugzilla.mozilla.org/attachment.cgi?id=8577204&action=edit

---

If I click the link to the attachment, I get the attachment edit page on Bugzilla.

I'd like for the email to contain a link to the MozReview URL for the review request.
Multiple people told me today that this is a papercut and the necessity for following multiple links is pretty annoying. Can we get this prioritized?
Flags: needinfo?(mcote)
This is a problem with Bugzilla's flagmail.  We submit a comment containing a link (admittedly a little obfuscated, to the attachment itself, which redirects to the review request), which generates bugmail, but this comment is not included in the separate "review requested" flagmail.

The "easy" fix is to include a link to the attachment in addition to (or maybe even instead of) the edit link.  A better fix would be to include a comment and clearly put the link to the review request in there.  I'll talk to the bmo team.
Flags: needinfo?(mcote)
Assignee: nobody → glob
Component: Extensions: MozReview Integration → Extensions: BMO
Summary: Email about review request from MozReview doesn't provide link to MozReview → emails about redirected attachment urls link to attachment-edit instead of the actual url (mozreview, github, etc)
Attached patch 1143132_1.patch β€” β€” Splinter Review
- adds a hook which allows extensions to override the attachment url in flagmail
- bmo ext adds a link to reviewboard/github/gdocs url

eg.
> Attachment 8591332 [details]: MozReview Request: Bug 35 - Switch paragraph order. r?dylan
> MozReview: https://reviewboard-dev.allizom.org/r/432/
> http://bz/1143132/attachment.cgi?id=8591332&action=edit
Attachment #8629237 - Flags: review?(dkl)
Comment on attachment 8629237 [details] [diff] [review]
1143132_1.patch

Review of attachment 8629237 [details] [diff] [review]:
-----------------------------------------------------------------

r=dkl
Attachment #8629237 - Flags: review?(dkl) → review+
To ssh://gitolite3@git.mozilla.org/webtools/bmo/bugzilla.git
   b46c86a..9528297  master -> master
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
fix bustage:

To ssh://gitolite3@git.mozilla.org/webtools/bmo/bugzilla.git
   9528297..eddd417  master -> master
Component: Extensions: BMO → Extensions
You need to log in before you can comment on or make changes to this bug.