816193 - Rev Dash: 404 not found diffs due to locale

Status: RESOLVED FIXED

(developer.mozilla.org Graveyard :: Dashboards, defect)

Description

[Florian Scholz (Open Web Docs)]

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

[[github robot]]

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