YAFLogo

BWG
  • BWG
  • 100% (Exalted)
  • YAF Lover Topic Starter
a year ago
Hi,

I'm looking at upgrading from v2.2.4.15 to v3.1.8. I've got v3.1.8 running on my development server, but one thing I have noticed is that embedded Tweets are no longer rendered correctly. It looks as though the html source for the Tweet is getting encoded, or something like that.

Is there a setting which needs enabling to make Tweets (and presumably other embeds such as YouTube) display correctly?

Thanks

BWG

Sponsor
tha_watcha
  • tha_watcha
  • 100% (Exalted)
  • YAF.NET Project Lead 🤴 YAF Version: 4.0.0 rc 2
a year ago
twitter is still embedded as bbcode extension. you can configure how the extension is displayed via Admin > Settings > BBCode Extensions > Twitter

Can you post a screenshot of the problem?

 

BWG
  • BWG
  • 100% (Exalted)
  • YAF Lover Topic Starter
a year ago

twitter is still embedded as bbcode extension. you can configure how the extension is displayed via Admin > Settings > BBCode Extensions > Twitter

Can you post a screenshot of the problem?

 

Originally Posted by: tha_watcha 

My apologies. The BBCode extension was not brought across (not sure if that is expected behaviour) when I upgraded locally from v2.2.4.15. However, when I recreate the BBCode extension the embeds work as expected. 

tha_watcha
  • tha_watcha
  • 100% (Exalted)
  • YAF.NET Project Lead 🤴 YAF Version: 4.0.0 rc 2
a year ago
No that not the expected behaviour, all extensions should exist after the upgrade. Did all extensions get deleted?
BWG
  • BWG
  • 100% (Exalted)
  • YAF Lover Topic Starter
a year ago
Just looked at this again and realised the BBCode admin page was set to show only 5 entries. 😂 When I change that the full list appears, although my original point about having to recreate the Twitter embed still seem to be valid.
tha_watcha
  • tha_watcha
  • 100% (Exalted)
  • YAF.NET Project Lead 🤴 YAF Version: 4.0.0 rc 2
a year ago

Just looked at this again and realised the BBCode admin page was set to show only 5 entries. 😂 When I change that the full list appears, although my original point about having to recreate the Twitter embed still seem to be valid.

Originally Posted by: BWG 

Thanks for the update. What gets displayed when you dont re-create the twitter bbcode?

BWG
  • BWG
  • 100% (Exalted)
  • YAF Lover Topic Starter
a year ago
 tweet.PNG You have insufficient rights to see the content.I've attached a screenshot of what appears when the BBCode for tweets is not recreated.

PS: Is it possible via the UI to disable the cookie notifcation screen? (I'm using Quantcast across the site and don't need / want an additional notification on the forum)

 

tha_watcha
  • tha_watcha
  • 100% (Exalted)
  • YAF.NET Project Lead 🤴 YAF Version: 4.0.0 rc 2
a year ago
Its odd, looks like the extension is not correctly updated on the upgrade. I have to check that.

Yes you can disable the cookie dialog via Host > Host Settings > Host Setup > Show Cookie Consent Popup message 

Zero2Cool
  • Zero2Cool
  • 100% (Exalted)
  • YAF Leader YAF Version: YAF 3.1.16
a year ago

Just looked at this again and realised the BBCode admin page was set to show only 5 entries. 😂 When I change that the full list appears, although my original point about having to recreate the Twitter embed still seem to be valid.

Originally Posted by: BWG 

Each time I do an upgrade, I have to fix the Twitter embed. For whatever reason, it ends up getting switched to one where it just says "loading tweet" or something and takes up the full screen with white space.

What I do is edit the BBCode and under "Replace RegEx" I put this in here.  Please note, this was probably removed due to a security issue, so take that for what it's worth. If not security issue, I am not sure why it was removed but I am sure it was removed for good reason.

<!-- BEGIN twitter embed --><div class=""><blockquote class="twitter-tweet"><a href="https://twitter.com/x/status/${statusID}"></a></blockquote> <script async src="https://platform.twitter.com/widgets.js" charset="utf-8"></script></div><!-- END twitter embed -->

Weird, the preview showed it properly, but after submitting it less than and greater than symbols get transposed. Oh well, here is what it should look like.

 image.png You have insufficient rights to see the content.

tha_watcha
  • tha_watcha
  • 100% (Exalted)
  • YAF.NET Project Lead 🤴 YAF Version: 4.0.0 rc 2
a year ago

Each time I do an upgrade, I have to fix the Twitter embed. For whatever reason, it ends up getting switched to one where it just says "loading tweet" or something and takes up the full screen with white space.

Originally Posted by: Zero2Cool 

Because your bbcode is completly different from the one thats included in yaf. You are using statusID of the tweet where the yaf one uses the entire tweet url. Also the white space issue is fixed in 3.19

Zero2Cool
  • Zero2Cool
  • 100% (Exalted)
  • YAF Leader YAF Version: YAF 3.1.16
a year ago

Each time I do an upgrade, I have to fix the Twitter embed. For whatever reason, it ends up getting switched to one where it just says "loading tweet" or something and takes up the full screen with white space.

Originally Posted by: tha_watcha 

Because your bbcode is completly different from the one thats included in yaf. You are using statusID of the tweet where the yaf one uses the entire tweet url. Also the white space issue is fixed in 3.19

Originally Posted by: Zero2Cool 

If I change to the one that now comes with YAF, then all the other embedded Tweets would error out, right? 

tha_watcha
  • tha_watcha
  • 100% (Exalted)
  • YAF.NET Project Lead 🤴 YAF Version: 4.0.0 rc 2
a year ago

Each time I do an upgrade, I have to fix the Twitter embed. For whatever reason, it ends up getting switched to one where it just says "loading tweet" or something and takes up the full screen with white space.

Originally Posted by: Zero2Cool 

Because your bbcode is completly different from the one thats included in yaf. You are using statusID of the tweet where the yaf one uses the entire tweet url. Also the white space issue is fixed in 3.19

Originally Posted by: tha_watcha 

If I change to the one that now comes with YAF, then all the other embedded Tweets would error out, right? 

Originally Posted by: Zero2Cool 

i build an regex to support both

 BBCodeTwitter.xml You have insufficient rights to see the content.

now the twitter bbcode can handle the status url or just the status id