Oh, thanks! Is it possible to do that without actually allowing Twitter logins?
Disable enable twitter logins and just provide twitter consumer key & twitter consumer secret.
3 Mi Piace
Tried it. It didn’t work. Embedded tweets still do not contain images.
Some long tweets with images don’t work
Shorter tweets with images works
It exists since the new twitter policy regarding links and images in the 140 characters count
6 Mi Piace
I’ve just submitted a PR which should fix this:
10 Mi Piace
@david - thank you! We do a lot of Twitter oneboxing and found the truncation very annoying.
4 Mi Piace
This is terrific - my forum users really need this.
Is this merged into a future release or something we have to do to modify in our existing installations?
Just upgrade on latest version, it should be good :
- via
yourforum.com/admin/upgrade - or SSH into your server and do these commands :
cd /var/discourse
git pull
./launcher rebuild app
I don’t think you need to add the twitter consumer key and twitter consumer secret in your admin settings but I’m only 99% sure
2 Mi Piace
Yep - it’s all merged into master and working great on my forum
I’m afraid you will need to add these keys to your site settings - the data is fetched via the twitter API rather than opengraph/oembed.
2 Mi Piace
It’s always the 1% left 
1 Mi Piace
I just pushed a small improvement for Twitter onebox. The onebox will now show like and retweet count.
11 Mi Piace
Twitter onebox with videos are really long on mobile. Here’s one I am adding to this post.
How it appears on mobile:
5 Mi Piace
@awesomerobot is there a simple CSS fix for 
4 Mi Piace
At the moment on mobile we set a max-height (70vh) and 100% width, and the video scales to fit. This works better for portrait oriented videos than landscape.
We can assume most videos will be landscape, but then we crop portrait oriented videos pretty severely as a trade-off.
There’s no simple fix for both without knowing the aspect ratio (Are we able to calculate the aspect ratio of these and maybe append a style based on that? the iframe seems to come with height/width).
4 Mi Piace
Yes, we can do that. This is the line where we build the iframe
What HTML structure would be most useful for you @awesomerobot? Something like
<div class='tweet-images'>
<div class='aspect-image-full-size' style='--aspect-ratio:#{width}/#{height};'/>
<iframe class='tweet-video' src='https://twitter.com/i/videos/blah' width='123' height='456' frameborder='0' allowfullscreen></iframe>
</div>
</div>
And then we can re-use the aspect-image-full-size logic which we use for images?
1 Mi Piace
Yeah I think that could work, with the aspect ratio there I should be able to adjust as needed.
2 Mi Piace
Sto cercando di capire come fare in modo che i tweet estratti da Zapier appaiano come un argomento in questo modo, ma ho dei problemi. Qualcuno può indicarmi la strada giusta? Dovrebbe funzionare se abilito semplicemente “onebox” nelle impostazioni?
3 Mi Piace
Esiste un modo per far sì che i tweet incorporati mostrino anche il tweet a cui si sta rispondendo? Ad esempio, se incorporo questo tweet https://twitter.com/AOC/status/1230329539924025345, in phpBB e vBulletin vengono visualizzati sia il tweet che a cosa sta rispondendo.
phpBB:
vB:
Invece in Discourse viene mostrato solo il tweet di risposta, il che spesso non ha senso senza il contesto del tweet originale
Ovviamente è abbastanza semplice pubblicare anche il tweet originale insieme a quello di risposta, ma la maggior parte degli utenti (e alcuni bot Twitter che utilizziamo) non sono abbastanza abili o coscienziosi da capire che devono farlo.
7 Mi Piace
Questo è un problema davvero molto complesso: il codice di incorporamento “ufficiale” di Twitter presenta due problemi:
- Non fornisce l’altezza, quindi gli argomenti potrebbero spostarsi durante il caricamento.
- Dovremmo eseguire i file JavaScript di Twitter su tutti i nostri siti.
Abbiamo aggirato il problema utilizzando un renderer personalizzato, ma purtroppo ciò significa che dobbiamo continuamente cercare di stare al passo con il codice di incorporamento ufficiale.
6 Mi Piace