Breadcrumb/add-on name link in file viewer/differ are 404s

VERIFIED FIXED in 6.1.0

Status

VERIFIED FIXED
8 years ago
3 years ago

People

(Reporter: krupa.mozbugs, Assigned: gkoberger)

Tracking

unspecified
6.1.0

Details

(URL)

(Reporter)

Description

8 years ago
steps to reproduce:
1. Load https://addons-next.allizom.org/en-US/firefox/files/compare/120631...120627/
2. Click on 'Silvermaaal for 4.44' in the breadcrumb

expected behavior:
review detail page loads

actual behavior:
since the review detail page now uses slug instead of review id, this link is a 404.
(Reporter)

Comment 1

8 years ago
Same holds true for

a) add-on name in file viewer/differ
Summary: Breadcrumb link in file viewer is a 404 → Breadcrumb link in file viewer/differ is a 404
(Reporter)

Updated

8 years ago
Summary: Breadcrumb link in file viewer/differ is a 404 → Breadcrumb/add-on name link in file viewer/differ are 404s
(Reporter)

Comment 2

8 years ago
You also need to fix 'back to review' link
(Assignee)

Updated

8 years ago
Assignee: nobody → gkoberger
(Assignee)

Comment 3

8 years ago
http://github.com/jbalogh/zamboni/commit/ad09b80
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
(Reporter)

Comment 4

8 years ago
verified at https://addons-next.allizom.org/en-US/firefox/files/browse/120538/
Status: RESOLVED → VERIFIED
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.