-
Notifications
You must be signed in to change notification settings - Fork 5
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
get_posts raises exception #2
Comments
It breaks at this line: https://github.com/richardvenneman/instagrammer/blob/master/lib/instagrammer/post.rb#L61 |
Hi @jankeesvw, thanks for the report. It seems Instagram, for reasons unknown, doesn't expose the post metadata as a JSON+LD object on all posts (https://www.instagram.com/p/BzFv6Z5oAUX/ in this case). For users this is also the case and I support this case by checking for the existence of this metadata before scraping. It appears we're going to have to do the same for posts as well. Will work on this asap! |
I can also make the patch if you want? |
@jankeesvw Sure! I'd be happy to accept a pull request 😄 |
- Lazy get data - Get page status before getting data, addresses #2
@jankeesvw I just released 0.2.1 which resolves this issue. |
Thanks! ✨ |
Raises:
Do you think I should patch this gem to make this work?
The text was updated successfully, but these errors were encountered: