Page MenuHomeDevCentral

Analyse The Guardian
ClosedPublic

Authored by Thibaut120094 on Jan 10 2017, 09:10.

Diff Detail

Repository
rSTG Source templates generator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Thibaut120094 created this revision.Jan 10 2017, 09:10
dereckson updated this revision to Diff 2073.Jan 11 2017, 04:56
dereckson edited edge metadata.

Expand tab to spaces

dereckson edited the test plan for this revision. (Show Details)Jan 11 2017, 04:59
dereckson edited the summary of this revision. (Show Details)

Looks good, but as long as T1091 isn't solved, to use http://www.theguardian.com isn't useful, as The Guardian web server now rewrites URL to https.

dereckson requested changes to this revision.Jan 11 2017, 06:50
dereckson added inline comments.
pages/index.dat
27

Probably best to use https:// here, or to solve T1091 first.

This revision now requires changes to proceed.Jan 11 2017, 06:50
dereckson updated this revision to Diff 2111.Jan 13 2017, 17:09
dereckson edited edge metadata.

+https in pages/index.dat

dereckson accepted this revision.Jan 13 2017, 17:09
This revision is now accepted and ready to land.Jan 13 2017, 17:09
This revision was automatically updated to reflect the committed changes.