Skip to content
This repository has been archived by the owner on Mar 31, 2022. It is now read-only.

Instagram embeds not wrapped in op-social tags #38

Open
lukecd opened this issue May 20, 2016 · 18 comments
Open

Instagram embeds not wrapped in op-social tags #38

lukecd opened this issue May 20, 2016 · 18 comments

Comments

@lukecd
Copy link

lukecd commented May 20, 2016

Hi, check out this post

https://picnic.ly/2016/05/cool-accessories-igshop/

It has a bunch of images that are not Instagram embeds and then a bunch of Instagram embeds. The non-IG images show up in the Instant Article, but the Instagram posts just don't show up.

I put the code from the WordPress editor here

http://pastebin.com/35DfUWWR

The converted Instant Article HTML with errors from the Facebook backend is here

http://pastebin.com/9fUPaCTC

Here's a screenshot showing some of the above HTML, this might be easier to read though as it has FB's syntax coloring going for it

http://imgur.com/2CnYmE9

Thanks!

Technical info

  • WordPress version: 4.5.2
  • allfacebook Instant Articles version:
@lukecd
Copy link
Author

lukecd commented May 20, 2016

Forgot to say that I'm using Version 0.7.1 of the plugin.

@Luehrsen
Copy link
Member

Are you working with the current version of the plugin?

Because I cannot replicate the problem. Even with the [embed] tags the code looks fine to me (on my server).
bildschirmfoto 2016-05-20 um 11 33 36

@Luehrsen Luehrsen added the bug label May 20, 2016
@lukecd
Copy link
Author

lukecd commented May 21, 2016

Hi, I'm on v 0.7.1 of the plugin. Have you tried testing with my full post?

http://pastebin.com/35DfUWWR

The only thing I can think of that's different with my version is the use of Thai characters, and the large number of embeds, but not sure that would cause any problems.

Is there any way I can test your version of the post with my setup (without actually making a post live?). Thanks.

@lukecd
Copy link
Author

lukecd commented May 21, 2016

ps: just confirmed with another post. This one is all Instagram embeds, and the instant article contains only text, no images

https://picnic.ly/2016/05/cute-girl-cool-style/

@Luehrsen
Copy link
Member

Give me a few hours to confirm. I want to find this bug. :D

@lukecd
Copy link
Author

lukecd commented May 23, 2016

Any luck replicating it?

Thanks!

@Luehrsen
Copy link
Member

Hey Luke,

no luck yet. Even with the full post replicated I get correct op-social tags.

bildschirmfoto 2016-05-27 um 15 40 18

@lukecd
Copy link
Author

lukecd commented May 28, 2016

Hmmm, sounds like a plugin conflict then, huh?

Think any of these could be causing a problem?

  • Social by Yoast
  • NextScripts: Social Networks Auto-Poster
  • WP Super Cache

I've got others running, but those are the only ones that stand out as potentially causing a problem.

Thanks!

@Luehrsen
Copy link
Member

I've got the Yoast Plugin and WP Super Cache running myself. The Auto Poster should not interfere with the content.

Have you had any luck identifying the issue?

@lukecd
Copy link
Author

lukecd commented Jun 14, 2016

Not yet ... but I haven't looked in a few days, got caught up on some other stuff. Hmmm I wonder if WPML might be causing a problem. Let me turn that off and try.

btw: do you know when advertising support will launch?

@lukecd
Copy link
Author

lukecd commented Jun 14, 2016

Hmmm I just tested w/o WPML and it's the same issue. The only thing that doesn't work is Instagram embeds. The posts just show up with text only, the IG embeds disappear.

@lukecd
Copy link
Author

lukecd commented Jun 21, 2016

Hi I just had another post come out that is a mix of instagram embeds and normal image embeds, and confirmed that the Instagram images do not show up, only the normal images. Thanks again.

https://picnic.ly/2016/06/5-fashion-bloggers/

@lukecd
Copy link
Author

lukecd commented Jul 12, 2016

hi ... i'm still trying to track this down. i thought it might be a conflict with the SiteOrigin PageBuilder plugin, but i disabled it, removed all the posts in FB and resent them ... and I still have the bug :( Just can't seem to get it to properly embed Instagram posts, they just get totally stripped from the Instant Article.

Thanks for adding support for Ads btw!

@lukecd
Copy link
Author

lukecd commented Jul 13, 2016

Hi ... So I've been manually converting over some posts and thought I'd show you one as an example.

Here's the code your plugin generates, when viewed in the Instant Article preview no Instagram images show

http://pastebin.com/cTcGxxYa

When I manually modify it to be like this

http://pastebin.com/eL8g5neg

Everything works perfectly. For now I'll manually fix things as I really want to launch this, but if you have any idea of why is causing the problem please let me know. Thanks!

btw: here's the original post

https://picnic.ly/2016/05/3d-tattoo-mania/

Thanks!

@lukecd
Copy link
Author

lukecd commented Aug 17, 2016

Hey ... I know you're super busy working on this plugin, but I just wanted to touch base about this issue and see how it's going? Thanks!

@lukecd
Copy link
Author

lukecd commented Sep 14, 2016

Hi ... Just a kinda-update about this issue. For the first time I had a post with a Twitter embed and was worried that I'd encounter the same problem as with Instagram embeds, but it worked perfectly. Not sure why I'm still having problems with IG embeds, my only idea is that maybe it has to do with the fact that my IG embeds are somehow getting wrapped in

tags and that's confusing your plugin? When I manually replace the embed code with the op-embed tag, I need to delete the

tags on my end.

Thanks again, other that this the plugin is great.

@Asinox
Copy link

Asinox commented Apr 2, 2017

Hello guys!, i'm having the same issue.. any info somewhere?

thank you!!

@Luehrsen
Copy link
Member

I think I just found the issue.

It went out with the last patch. Can you test and report please? :) Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants