Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hi,
I was wondering if there's any way to add moderation to published tweets, ie, tweets fetched by the cron will only be displayed after an admin/authorized user approves them.
If not, how should I address this (I am a Drupal noob, so any help is welcome)
cheers
Comment | File | Size | Author |
---|---|---|---|
#15 | twitter-n1916366-15.patch | 14.82 KB | leewillis77 |
#12 | twitter-n1916366-12.patch | 15.63 KB | DamienMcKenna |
Comments
Comment #1
juampynr CreditAttribution: juampynr commentedThis issue could lead to some sort of moderation #1850542: Add ability to retrieve favorite tweets from a Twitter account, but it has not been implemented yet.
If you really want to be able to moderate each tweet (or been able to delete tweets), at the moment there is no such feature. I wonder if adding a Delete link would help.
Comment #2
tiagojsag CreditAttribution: tiagojsag commentedThis patch should do the trick. This is my first time contributing to a Drupal module, so please be gentle when pointing out all the noob mistakes I made ;)
Comment #3
tiagojsag CreditAttribution: tiagojsag commentedComment #4
leewillis77 CreditAttribution: leewillis77 commentedThis patch applied (With some offsets, but otherwise cleanly), and works perfectly for our needs.
Comment #5
leewillis77 CreditAttribution: leewillis77 commentedThe attached patch is a re-roll of tiagojsag's patch against current dev.
It removes some trailing whitespace errors, and also renames the permission from "moderate posts" to "moderate tweets" which seems more appropriate.
Comment #6
DamienMcKennaTriggering the testbot.
Comment #7
DamienMcKennaTriggering the testbot.
Comment #10
DamienMcKennaRerolled.
Comment #12
DamienMcKennaFixed the dupe update script.
Comment #15
leewillis77 CreditAttribution: leewillis77 at Upbeat Productions commentedRe-rolled the patch against current dev. since the previous patch no longer applies. Fairly manual process - so would be good to get some more eyes on this although it is working for me so far.
Comment #16
dani3lr0se CreditAttribution: dani3lr0se at Hook 42 commentedThe patch applies for me locally. What is happening for me is, no tweets are publishing unless I check the "publish" box next to the tweets, which I see even if the "moderated" box is checked. Is this normal? So no matter if the "Moderated" box is checked after I add my twitter account, I still need to check a publish box to publish tweets. Sorry if I have misunderstood this.
My process:
1) Added my twitter account/creds, etc.
2) checked "tweets", "mentions" and "moderated"
3) viewed the "tweets" tab to see tweets after running cron
4) needed to check the "publish" box next to a tweet/s in order for tweets to publish. I had to do this regardless of whether or not "moderated" was checked.
Also, I realize this is 2 years old and may need to be updated/rerolled.
Thanks for other thoughts, etc. I'll continue to play around with it and make sure I'm doing things correctly.
Comment #17
oadaeh CreditAttribution: oadaeh at Hook 42 commentedThe patch still applies cleanly, though it does add an extra line at the end of twitter.install. I did not test functionality or look at the code changes.
Comment #18
oadaeh CreditAttribution: oadaeh at Hook 42 commentedOkay, I ran into some PHP notices that are a result of the patch, so more work needs to be done with it:
Comment #19
oadaeh CreditAttribution: oadaeh at Hook 42 commentedThose PHP notices might be because I didn't run any updates. So feel free to revert the status, if updating is needed.