Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 9103016: Fix url-based rewrites (Closed)

Created:
Jan. 2, 2013, 2:58 p.m. by Felix Dahlke
Modified:
Nov. 8, 2013, 8:04 a.m.
Reviewers:
Wladimir Palant
Visibility:
Public.

Description

Before this change, "/en/android-about" would have been rewritten to "/en/?browser=android", which is not correct.

Patch Set 1 #

Patch Set 2 : Made browser rewriting work for /en/ etc. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -3 lines) Patch
M modules/adblockplusorg/files/adblockplus.org View 1 3 chunks +3 lines, -3 lines 0 comments Download

Messages

Total messages: 2
Felix Dahlke
Jan. 2, 2013, 2:59 p.m. (2013-01-02 14:59:00 UTC) #1
Wladimir Palant
Jan. 9, 2013, 10:45 a.m. (2013-01-09 10:45:40 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld