Closed (fixed)
Project:
Drupal.org content
Component:
Planet Drupal
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
8 Sep 2011 at 08:17 UTC
Updated:
18 Nov 2015 at 10:54 UTC
Jump to comment: Most recent
Blog name: Leow Kah Thong
Drupal-Planet tagged feed URL: http://kahthong.com/taxonomy/term/4/feed
Taxonomy term page: http://kahthong.com/tags/drupal
Comments
Comment #1
BrockBoland commentedFrom the Planet Drupal guidelines:
Your feed must be Planet-specific.Your feed should have at least a few posts already so we can evaluate the content and make sure it is appropriate for the feed. More than three posts is preferred.Your feed must pass source code validation since Drupal's aggregator module is very strict about parsing feed source code.Comments must be enabled. We highly recommend, but do not require, that anonymous comments be enabled for the content being aggregated.The content looks mostly good, but review the posts that are not welcome:
I think that the post about URL shortener falls a bit into the second and second-to-last items. If you could remove the Drupal Planet tag from that one, I think this is ready to RTBC.
Comment #2
ktleow commentedHey BrockBoland,
Thanks for getting back to me on this long-lost request.
I thought this request had fallen on deaf ears.
Anyway, I've removed that URL shortener post from the 'drupal' taxonomy term.
Anything else? :-)
Comment #3
BrockBoland commentedLooks good to me!
Comment #4
cweagansThe dates on a couple of your posts are off and will mess with the aggregator a bit (the post will just sit at the top of planet until that date passes). This is not okay, so back to needs work.
Comment #5
ktleow commentedHi cweagans,
Thanks for reviewing, but what do you mean by the dates are off and will mess with the aggregator?
Comment #6
cweagansEmbed Drupal node form anywhere
Posted on Tue, 10/01/2012 - 13:24
Display Drupal menu local task tab according to node content type
Posted on Sun, 08/01/2012 - 16:38
For some reason, you have posts that have future dates for the publish dates. Aggregator will pull those in with the dates that you've given them and display the posts in reverse chronological order, which means that your posts will just sit at the top of aggregator for months. If this happened, we would remove you from planet, so let's fix it before you're added :)
Comment #7
BrockBoland commentedLooks like European-style dates vs. American style dates. I'm looking at the feed itself and don't see these dates, though: has it been fixed?
Comment #8
cweagansOh, you know what? You're right. Sorry, it was late. I'll add this later today if somebody doesn't beat me to it.
Comment #9
ktleow commentedI'm from Malaysia, we're using UK style dates DD/MM/YYYY.
I'm pretty sure this is not a problem for the Aggregator, as it reads the date from feed. For example:
Thanks for looking into this issue guys, appreciate that :)
Comment #10
cweaganshttp://drupal.org/aggregator/sources/1004
Sorry about the delay!
Comment #11
ktleow commentedSuper thanks! Shall start writing constructive Drupal tutorials with more themes when I'm free!
Comment #13
dddave commentedThe feed from Leow Kah Thong seems to be broken, due to an error "Not well-formed (invalid token)" on line 1.Feed suspended for now.
Comment #14
dddave commentedNo idea why I assigned this to me.
Comment #15
dddave commentedseems to be fixed.