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

feat: fortune parser #84

Merged
merged 3 commits into from
Jan 24, 2017
Merged

feat: fortune parser #84

merged 3 commits into from
Jan 24, 2017

Conversation

janetleekim
Copy link
Contributor

For some reason, the dek doesn’t appear in the local version of the
article I selected. I tried parsing the meta tag containing
og:description but it’s not working, and the description is slightly
longer than the dek in the original article.

I’m not sure why, but for the lead image, the meta tag for og:image is
not parsing the image url.

:(

janetleekim and others added 2 commits January 23, 2017 16:30
For some reason, the dek doesn’t appear in the local version of the
article I selected. I tried parsing the meta tag containing
og:description but it’s not working, and the description is slightly
longer than the dek in the original article.

I’m not sure why, but for the lead image, the meta tag for og:image is
not parsing the image url.

:(
@adampash adampash force-pushed the feat-fortune-extractor branch from 3cedce4 to 77ef462 Compare January 24, 2017 00:30
Copy link
Contributor

@adampash adampash left a comment

Choose a reason for hiding this comment

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

Fortune redesigned, so redid the selectors on this one.

@adampash adampash merged commit ad29acd into master Jan 24, 2017
@adampash adampash deleted the feat-fortune-extractor branch January 24, 2017 00:47
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