Closed Bug 1229733 Opened 6 years ago Closed 6 years ago

Fix rendering of autoland actions when the landing repository has trailing slashes

Categories

(MozReview Graveyard :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mdoglio, Assigned: mdoglio)

Details

Attachments

(2 files)

When an autoland repository url has a trailing slash the treeherder results don't show up and the link to code landed on mercurial is not correct.
https://reviewboard-dev.allizom.org/r/527/ has an example of both these issues
mozreview: add cedar to the list of treeherder repositories; r?dminor
Attachment #8694677 - Flags: review?(dminor)
mozreview: remove trailing slashes from autoland repository url (bug 1229733); r?dminor
Attachment #8694678 - Flags: review?(dminor)
Attachment #8694677 - Flags: review?(dminor) → review+
Comment on attachment 8694677 [details]
MozReview Request: mozreview: add cedar to the list of treeherder repositories; r=dminor

https://reviewboard.mozilla.org/r/26829/#review24283
Comment on attachment 8694678 [details]
MozReview Request: mozreview: remove trailing slashes from autoland repository url (bug 1229733); r=dminor

https://reviewboard.mozilla.org/r/26831/#review24285
Attachment #8694678 - Flags: review?(dminor) → review+
Comment on attachment 8694677 [details]
MozReview Request: mozreview: add cedar to the list of treeherder repositories; r=dminor

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/26829/diff/1-2/
Attachment #8694677 - Attachment description: MozReview Request: mozreview: add cedar to the list of treeherder repositories; r?dminor → MozReview Request: mozreview: add cedar to the list of treeherder repositories; r=dminor
Attachment #8694678 - Attachment description: MozReview Request: mozreview: remove trailing slashes from autoland repository url (bug 1229733); r?dminor → MozReview Request: mozreview: remove trailing slashes from autoland repository url (bug 1229733); r=dminor
Comment on attachment 8694678 [details]
MozReview Request: mozreview: remove trailing slashes from autoland repository url (bug 1229733); r=dminor

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/26831/diff/1-2/
Assignee: nobody → mdoglio
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Product: Developer Services → MozReview
You need to log in before you can comment on or make changes to this bug.