I made changes to YVoice folder #157

Merged
merged 5 commits into from Mar 16, 2012

2 participants

@houyhnhnm

I renamed the folder YVoices as yahoo voice is a particular service and yahoo contributor was changed to YVoices. I kept the three tables in the prior folder but added two more. One is YVoices.fans that gets fans from a user's page and favorites, that gets favorites from a user's page

they need null condition testing, but should work for all other cases

houyhnhnm added some commits Feb 6, 2012
@houyhnhnm houyhnhnm Merge remote-tracking branch 'upstream/master' a06fdd6
@houyhnhnm houyhnhnm Merge remote-tracking branch 'upstream/master' ba29205
@houyhnhnm houyhnhnm The prior yvoice folder should be deleted as the name is improper. Th…
…is folder contains the same tables as before with two added ones,

YVoices.fans that allows you to get the fans on a users page
and
YVoices.favorites that allow you to get the favorites on a users page

They need testing for the null condition but work in all others.
fba0b47
@houyhnhnm houyhnhnm Deleting yvoice collection because the yvoices table is appropriately…
… named and expanded
13e6c41
@houyhnhnm houyhnhnm The recent articles table displayed an error off of one users table s…
…tructure, the problem arrived because even though in all other cases I checked the table produced the proper output, in this particular users case, the yql query return's structure was uniquely different. so, I added basic conditionals to test throughout the structure of the yql query before I attached content from it to my xml structure.

I did the same for all the tables in the YVoices family
c7522c2
@pjdonnelly pjdonnelly merged commit ae9c75d into yql:master Mar 16, 2012
@pjdonnelly
Official YQL Community Repository member

Can you lower case the directory name to yvoice - Thanks

@houyhnhnm

I did it the only way I knew how mr. Donnelly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment