Closed Bug 663149 Opened 13 years ago Closed 13 years ago

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

Categories

(addons.mozilla.org Graveyard :: Admin/Editor Tools, defect)

Product:
addons.mozilla.org Graveyard
Component:
Admin/Editor Tools
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: krupa.mozbugs, Assigned: gkoberger)

References

(
URL
)

Details

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.
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
Summary: Breadcrumb link in file viewer/differ is a 404 → Breadcrumb/add-on name link in file viewer/differ are 404s
You also need to fix 'back to review' link
Assignee: nobody → gkoberger
http://github.com/jbalogh/zamboni/commit/ad09b80
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
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.