Rev Dash: 404 not found diffs due to locale

RESOLVED FIXED

Status

RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: fscholz, Unassigned)

Tracking

Details

(Reporter)

Description

6 years ago
Kinda tricky to reproduce but let me try: 

1) Choose "All locales" as the filter option

2) Make sure, you are viewing the en-US version of the dashboard 
   (https://developer.mozilla.org/en-US/dashboards/revisions)

3) Look for slugs which aren't under the en-US locale.
  e.g. "Empacotando_Extensões" Nov 28, 2012 11:04

4) See that the wrong compare Url is loaded:
https://developer.mozilla.org/en-US/docs/Empacotando_Extens%C3%B5es$compare?from=221456&to=331801&raw=1

5) Change your revision dashboard view to the Portuguese version:
   (https://developer.mozilla.org/pt-PT/dashboards/revisions)

6) See that now the correct compare url for "Empacotando_Extensões" is loaded. (pt-PT in the url of step 4)

Note, this does not seem to happen with all revisions but it seems like there is a wrong relation between the locale I'm viewing the dashboard and the locale of the revision *sometimes*.

Comment 1

6 years ago
Commits pushed to master at https://github.com/mozilla/kuma

https://github.com/mozilla/kuma/commit/145831aab0ee0a6d534042ca550e4c0418c150a1
Fix bug 816193: Correct locale in rev dash urls, PEP8 fixes

https://github.com/mozilla/kuma/commit/8000f4073bf83183ef0e3aabd6ee3978ee7e192b
Merge pull request #1001 from Elchi3/dash-404locale-816193

Fix bug 816193: Correct locale in rev dash urls, PEP8 fixes

Updated

6 years ago
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.