-
Notifications
You must be signed in to change notification settings - Fork 33
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
Including "Article" attributes #18
Comments
I'm very liberal when it comes to pull requests. So any contributions would be greatly appreciated! I'd like to clear up what you mean by "Article" attributes and "author"? |
Per the Open Graph Protocol , they refer to an article |
Right now the module will (should) only return OG meta tag data. So as it stands you should be getting the Number 3 on the Todo list is to use fallback data if an OG tag isn't present (document title, etc). I haven't gotten to this yet, but I wouldn't mind a PR on this! :) |
The I'll work on getting the author, first through meta data ( |
@ryanjyost could you provide a URL that uses the article OG type. This way I can get a better example. |
Here's one from the Washington Post, though the more I delve into this, the more it seems the Getting the author, if that's still considered a worthwhile addition to this module, may be best addressed through finding Sorry for the confusion! |
First, just want to quickly say thanks for this great module.
For the project I'm working on, I'd like to display the author. This isn't strictly "og", but should be simple addition? If you're against including for this project, I'll try on my own. Thanks, and happy to submit pull if/when I add the feature (would be my first, I'm a novice relatively speaking).
The text was updated successfully, but these errors were encountered: