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

Re-activate and Update Techmeme.com.xml #10271

Merged
merged 3 commits into from
Jun 10, 2017
Merged

Re-activate and Update Techmeme.com.xml #10271

merged 3 commits into from
Jun 10, 2017

Conversation

cschanaj
Copy link
Collaborator

@cschanaj cschanaj commented Jun 9, 2017

@cschanaj cschanaj mentioned this pull request Jun 9, 2017
Closed
@jeremyn
Copy link
Contributor

jeremyn commented Jun 9, 2017 •
edited
Loading

  • Put all five problem subdomains in the top comment under the single problem type Invalid certificate
  • Add a target and rule for http://search.techmeme.com here like this:
    <target host="search.techmeme.com" />
 
    <rule from="^http://search\.techmeme\.com/" to="https://www.techmeme.com/search" />

@jeremyn jeremyn self-assigned this Jun 9, 2017
@cschanaj cschanaj closed this Jun 10, 2017
@cschanaj cschanaj reopened this Jun 10, 2017
@cschanaj
Copy link
Collaborator Author

done. thanks!!

@jeremyn
Copy link
Contributor

jeremyn commented Jun 10, 2017

Thanks for making the changes, but search.techmeme.com should be in the top comment too, so people know what the special rule is for.

@jeremyn jeremyn merged commit 5c4155f into EFForg:master Jun 10, 2017
@jeremyn
Copy link
Contributor

jeremyn commented Jun 10, 2017

Thanks again, merged.

@jeremyn jeremyn removed their assignment Jun 10, 2017
@cschanaj cschanaj deleted the techmeme.com branch June 10, 2017 15:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
top-100k
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cschanaj @jeremyn @Hainish