Skip to content

Bug 1310075 - Tests for qq.com. r=Gijs#316

Merged
gijsk merged 1 commit intomozilla:masterfrom
evanxd:bug-1310075
Nov 9, 2016
Merged

Bug 1310075 - Tests for qq.com. r=Gijs#316
gijsk merged 1 commit intomozilla:masterfrom
evanxd:bug-1310075

Conversation

@evanxd
Copy link
Copy Markdown
Contributor

@evanxd evanxd commented Nov 9, 2016

@gijsk ,

I've added tests for qq.com. (the source.html is from 1)
Could you help review the patch?

Thanks.

@@ -0,0 +1,6 @@
{
"title": "DeepMind新电脑已可利用记忆自学 人工智能迈上新台阶_科技_腾讯网",
"byline": null,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should we file a followup issue to detect a byline, or is there no author (clearly/semantically) specified? I'm afraid I can't speak/read Chinese, and Google Translate was not very helpful. Based on its "translation", it looks to me like there's an author at the end of the article(编译/林靖东)but I can't see any specific markup around that, and I don't know if the same information is in some of the metadata elsewhere.

@gijsk gijsk merged commit 1b694cf into mozilla:master Nov 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants