jdolitsky / AppDotNetPHP

PHP library for the App.net Stream API
34 stars 19 forks source link

Add functions to handle User modification #31

Closed cdn closed 11 years ago

cdn commented 11 years ago

A few new functions for user profile data mods;

1 handles the JSON PUT for name, locale, timezone and description,

another 2 handle the avatar and cover images (these two call the fourth to do general image POSTs)

jdolitsky commented 11 years ago

@cdn hey i just committed @neuroscr 's last pull request for response envelope methods and now I cant automatically merge... Could you clone the latest copy then add your changes? @hxf148 can you then do the same after @cdn is done? Thanks a bunch guys. Let me know if there's a better way to handle this

jdolitsky commented 11 years ago

wait, actually that last commit was just for the streaming branch. What's the issue?

cdn commented 11 years ago

I've only been playing with the master branch (trunk in other circumstances) ... Thursday's API roll out of User change things didn't exist, so I wrote something, which probably warrants checking over :)

Also, httpReq() lacked PUT support ...

jdolitsky commented 11 years ago

awesome, well done! but was your branch up-to-date before you made changes?

cdn commented 11 years ago

I believe so.

Whatever mess my fork was in, I merged with master from here and then added Sunday/Monday’s changes.

That is how it’s supposed to work, right? :)

cdn commented 11 years ago

I've saved @hxf148 the trouble on this occasion; refactored his code slightly

ghost commented 11 years ago

Much appreciated! I'll make a fork

On Wednesday, October 31, 2012, cdn wrote:

I've saved @hxf148 https://github.com/hxf148 the trouble on this occasion; refactored his code slightly

— Reply to this email directly or view it on GitHubhttps://github.com/jdolitsky/AppDotNetPHP/pull/31#issuecomment-9939485.

Harold http://hxf148.com

cdn commented 11 years ago

I later made @neuroscr's change to rename your function :)