Skip to content
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

wallabag retrieves only parts of multi-part feature article #2153

Closed
solarchemist opened this issue Jun 17, 2016 · 4 comments
Closed

wallabag retrieves only parts of multi-part feature article #2153

solarchemist opened this issue Jun 17, 2016 · 4 comments

Comments

@solarchemist
Copy link

Issue details

http://www.houstonchronicle.com/nasa/adrift/1/
This is a seven part feature article with lots of text and plenty of graphics.
Wallabag managed to get the title, and a subset of the graphics (including their captions!).
But sadly, none of the body text.

Environment

  • wallabag version (or git revision) that exhibits the issue: 2.0.5
  • How did you install wallabag? Via git clone or by downloading the package? git clone
  • Last wallabag version that did not exhibit the issue (if applicable):
  • php version: 5.6.22
  • OS: Debian Jessie
  • type of hosting (shared or dedicated): dedicated
  • which storage system you choose at install (SQLite, MySQL/MariaDB or PostgreSQL): MySQL

Steps to reproduce/test case

Paste URL into "new entry" box, or add via Firefox addon.

@Strubbl
Copy link
Contributor

Strubbl commented Jun 17, 2016

see fivefilters/ftr-site-config#178 for fix

@solarchemist
Copy link
Author

I recently upgraded to Wallabag 2.0.7 (stack is now PHP 5.6.24, Debian Jessie, dedicated server, MySQL), and this issue still exists.

Thanks for the pointer to your pull request, but it seems not much has changed since your comment that only images and captions show up in Wallabag (but no article text). I noted the same behaviour.

But in any case, I think this kind of multi-part feature article is an edge case, and I'm very happy with Wallabag. Keep up the good work!

@j0k3r
Copy link
Member

j0k3r commented Sep 4, 2016

@chepec yep, the PR hasn't been merged so the problem is still here

@j0k3r
Copy link
Member

j0k3r commented Sep 4, 2016

@chepec will be available in the next release 🚀

@j0k3r j0k3r closed this as completed Sep 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants