-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Links that are not working correctly. [post them here] #3
Comments
http://bits.blogs.nytimes.com/2014/04/26/writing-in-a-nonstop-world
|
There's a bug with nytimes and request library. Not much I can do here. |
Trailing white space in title from: |
Thanks! Fixed that. |
First of all, thanks for doing this! The jsdom one was way too slow for my purposes and I've become increasingly frustrated by it. The performance improvements are off the charts! A couple minor issues popped up when I migrated to your version: The h1 tag is coming back inside a p tag. Also, In node-readability I believe these codes (& #8217; & #039; & quot; & #8220; & #8221;) were translated automatically. Up to you if you want to mimic that behaviour or not. |
Thanks! I'll look into it. |
Another issue I'm afraid... http://www.bbc.co.uk/sport/0/football/29053651 The first sentence in the article is not included in the result. |
Hello, I get this has launched a stripped-down version of its app aimed at the huge potential user base in emerging markets. The social media giant that it was testing the Android app in January, and was now rolling it out across countries in Asia, followed by parts of Latin America, Africa and Europe. (and more...)Basically, its cleaning out anchor tags within main article. |
A simple tag is not showing links when getting through article.content: |
http://tech.sina.com.cn/t/2016-04-20/doc-ifxrpvcy4243221.shtml error when handleing this link |
If you find any links that node-read cannot correctly parse, please post them here.
The text was updated successfully, but these errors were encountered: