I have attempted to follow the steps for creating the Twitter feed widget, as found here: https://twitter.com/settings/widgets
When you click "create new", you are taken to a separate "publish.twitter.com" subdomain in which you don't seem to be able to persistently a new widget. You can produce an embed code.
This embed code is very similar to the current embed code in the module, but there is a Javascript error when passing the widget_id for *new* sites that now seem to not be able to produce a widget id.
I'll be submitting a patch that removes the required form value for new Twitter feed instances.
Comment | File | Size | Author |
---|---|---|---|
#5 | twitter_block-2751845-5.patch | 4.25 KB | ckoppenhaver |
| |||
#2 | twitter_block-issue_2751845-1.patch | 2.62 KB | nerdstein |
|
Comments
Comment #2
nerdsteinA patch that makes widget ID not required.
Comment #3
naveenvalechaThanks for initial patch.
This needs hook_update as well.
Comment #4
Jorge Navarro CreditAttribution: Jorge Navarro commentedThe widget-data-id is no longer available when creating a twitter widget. Maybe the field should be removed. Creating the block without the id works fine.
For lists now we need to add [user]/lists/[list-name].
Comment #5
ckoppenhaver CreditAttribution: ckoppenhaver at Phase2 commentedI have added to the patch to take care of the form validation looking for the widget_id also updated the schema to allow non unique widget_ids in the database.
Comment #6
naveenvalecharunning tests
Comment #7
EsmeraldaUBM CreditAttribution: EsmeraldaUBM commentedTested and reviewed. Works with widget and without widget ID.
Comment #8
JosueRodrigu3z CreditAttribution: JosueRodrigu3z as a volunteer and commentedAny update on when this would be merged? We are having this issue at Yale University and quite a few users in need of this update.
Comment #9
t.a. barnhart CreditAttribution: t.a. barnhart commentedthanks for doing this! Twitter doesn't really understand social very well, i don't think.
(could a note be added to the module's page noting that new users need to patch the module? hopefully others won't waste time before clicking to the Issues page as i did. thank you!)
Comment #10
jrearickI'm with JosueRodrigu3z and would like to see this at least merged into a dev branch so we don't have to mange this patch. #5 seems to work well for us. As far as I know, there is no workaround to get this to work with new widgets without applying this patch.
Comment #11
hamuchen CreditAttribution: hamuchen commented#5 works great, this module doesn't work (for new installs) without it => changed prio and category.
Comment #12
Anonymous (not verified) CreditAttribution: Anonymous commented#In the meantime, maybe somebody could explain to this numpty how to apply a patch?
Comment #13
naveenvalechahttps://www.drupal.org/patch/apply
Comment #14
upperholme CreditAttribution: upperholme commentedI'm not clear as to why this patch has not been merged into a release some six months after the patch was produced. The module as it stands doesn't appear to work at all, for reasons as described above, so new users, myself included waste time installing a broken module. This module has a status of being 'actively maintained' but that doesn't appear to be the case. Maybe the maintainer has walked away?
Comment #15
apatsekin CreditAttribution: apatsekin commentedSo the module (stable and dev versions) for Drupal 8 is now completely useless. It requires WidgetID which was removed by Twitter a half-year ago.
Comment #16
hassebasse CreditAttribution: hassebasse commentedAbsolutly correct apatsekin, the module is useless as there is no WidgetID anymore and it should be mentionned so noone is spendning time with it as I did yesterday. I added the Twitter code directly into a bloc and it works perfectly fine.
Here is an explanation by andypiper at Twitter Staff.
https://twittercommunity.com/t/how-can-i-get-a-twitter-widget-id/68917/17
All the best
Hans
Comment #17
pheski CreditAttribution: pheski as a volunteer commented@hassebasse -
I've not been successful yet at adding the twitter code (for a timeline) into a custom block.
I got the embed code from Twitter, created a custom block (full html) and pasted the embed code in. The block shows me the code but not the timeline. Any help would be greatly appreciated.
Comment #18
hassebasse CreditAttribution: hassebasse commentedHi
Probably you have just pasted the code in just as you paste any text and then it will transform to and show up as text.
In Full HTML you will find a a button named Source. Click on that button, paste the code, click on the button again so you get out form the source and then you save the block. Now the HTML code from Twitter is saved as HTML code in the block.
You can use the Source function to modify almost any HTML and CSS code manually.
All the best
Hans
Comment #19
pheski CreditAttribution: pheski as a volunteer commented@hassebasse
Thank you x 3:
Your karma account has been credited. :-)
Comment #20
hassebasse CreditAttribution: hassebasse commentedHi
It's a shame that you feel you have to mention n° 3, isn't it, but I do understand your feeling. I've fought with Drupal for 2 years now and I remember very well how desperate I have been when noone have helped me when I've been stuck.
Thanks for karma !
Have a good evening (evening at least here in France)
Hans
Comment #21
pheski CreditAttribution: pheski as a volunteer commentedAs a follow-up, I tried to do the same thing on a Drupal 7 site and was stuck for a while. It kept stripping out the second (script) piece of my embed code.
The solution I found was to edit the CKEditor configuration -> cleanup and output -> advanced content filter (it was disabled by default) -> automatic
and make sure this line is there: config.allowedContent = true;
Hope this helps someone else.
Comment #22
Gixxernutter CreditAttribution: Gixxernutter commented+1 on twitter_block-2751845-5.patch - works great - thanks
Comment #23
hassebasse CreditAttribution: hassebasse commentedGixxernutter, but why using the module at all? Why not just putting the code into a block?
All the best
Hans
Comment #24
Gixxernutter CreditAttribution: Gixxernutter commentedHi Hans,
I use the Twitter Block module as it allows you to specify the number of tweets to display.
Peter
Comment #25
hassebasse CreditAttribution: hassebasse commentedHi Peter
Aha, thanks.
Hans
Comment #26
Ludo.RI confirm #5 patch allows me to use this module without a widget ID.
Comment #27
Gixxernutter CreditAttribution: Gixxernutter commentedJust tried to use this block again, you can't obtain a widget ID key as others have said, so the module won't work.
For easy reference, use this code in a block :
<a class="twitter-timeline" data-dnt="true" data-tweet-limit="1" href="https://twitter.com/YOUR TWEET HANDLE">Tweets by YOUR TWEET HANDLE</a> <script async src="//platform.twitter.com/widgets.js" charset="utf-8"></script>
data-tweet-limit="1" : displays the number of tweets that you want to show
Peter
Comment #28
rooby CreditAttribution: rooby commentedSeems like something that needs to be committed
Comment #29
jannis CreditAttribution: jannis commentedRTBC +1
Comment #30
EsmeraldaUBM CreditAttribution: EsmeraldaUBM commentedConfirmed as working RTBC +1
Comment #31
ericpoir CreditAttribution: ericpoir commentedI tested the patch from comment #5 and it seems to be working well.
Comment #32
CiviFirst John CreditAttribution: CiviFirst John commentedPlease merge this!