Skip to content
This repository has been archived by the owner on Nov 6, 2023. It is now read-only.

Update Time_Inc.xml #9732

Merged
merged 2 commits into from
May 2, 2017
Merged

Update Time_Inc.xml #9732

merged 2 commits into from
May 2, 2017

Conversation

cschanaj
Copy link
Collaborator

@cschanaj cschanaj commented May 1, 2017

Related: #9672

@@ -1,120 +1,56 @@
<!--
Disabled by https-everywhere-checker because:
Fetch error: http://timeinc.net/ => https://timeinc.net/: (7, 'Failed to connect to timeinc.net port 443: Connection timed out')
Other Time Inc rulesets:

- Customersvc.com.xml
- Entertainment_Weekly.xml
- Fortune.com.xml
- People.com.xml
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please also link this to Time.com.xml

<test url="http://fonts.timeinc.net/ti/fortune/webfonts.css?ver=1.0.00" />
<target host="git.timeinc.net" />
<target host="img.timeinc.net" />
<test url="http://img.timeinc.net/tii/omniture/h/config/time_s_code_multivideo.js" />
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • http://img5.timeinc.net/keynote/above_the_fold_1x1.gif

@cschanaj
Copy link
Collaborator Author

cschanaj commented May 1, 2017

thanks!

@J0WI J0WI merged commit cfa58eb into EFForg:master May 2, 2017
@cschanaj cschanaj deleted the cschanaj-timeinc.net branch May 3, 2017 00:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants