Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(4663)

Issue 294020043: issue 4813: replace urchin.js with analytics.js and fix redundant hostnames (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
8 years ago by fedelogy
Modified:
8 years ago
Reviewers:
phileholmes
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

issue 4813: replace urchin.js with analytics.js and fix redundant hostnames The change to the htaccess file should be tested manually on the server before pushing this patch to staging.

Patch Set 1 #

Total comments: 2

Patch Set 2 : Fix the RewriteRule (thanks to Phil) #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -6 lines) Patch
M Documentation/web/server/lilypond.org.htaccess View 1 1 chunk +4 lines, -0 lines 0 comments Download
M scripts/build/website_post.py View 1 chunk +9 lines, -6 lines 0 comments Download

Messages

Total messages: 5
fedelogy
The change to the htaccess file should be tested manually on the server before pushing ...
8 years ago (2016-03-30 07:23:29 UTC) #1
PhilEHolmes
Update: https://codereview.appspot.com/294020043/diff/1/Documentation/web/server/lilypond.org.htaccess File Documentation/web/server/lilypond.org.htaccess (left): https://codereview.appspot.com/294020043/diff/1/Documentation/web/server/lilypond.org.htaccess#oldcode19 Documentation/web/server/lilypond.org.htaccess:19: # Deny following of symlinks by robots. This ...
8 years ago (2016-03-30 10:20:52 UTC) #2
fedelogy
https://codereview.appspot.com/294020043/diff/1/Documentation/web/server/lilypond.org.htaccess File Documentation/web/server/lilypond.org.htaccess (left): https://codereview.appspot.com/294020043/diff/1/Documentation/web/server/lilypond.org.htaccess#oldcode19 Documentation/web/server/lilypond.org.htaccess:19: # Deny following of symlinks by robots. On 2016/03/30 ...
8 years ago (2016-03-30 11:08:28 UTC) #3
PhilEHolmes
Try now. Problem was with the website build system. If you edit the _real_ .htaccess, ...
8 years ago (2016-03-30 11:35:42 UTC) #4
fedelogy
8 years ago (2016-03-30 14:52:42 UTC) #5
Fix the RewriteRule (thanks to Phil)
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b