Dynamic requests #72

Merged
michaelhelmick merged 7 commits from dynamic_requests into master 2012-04-09 18:00:20 -07:00
michaelhelmick commented 2012-03-31 17:16:48 -07:00 (Migrated from github.com)
No description provided.
michaelhelmick commented 2012-04-06 08:49:11 -07:00 (Migrated from github.com)

All is good to go. My next plan is to clean up internal functions to acutally use self._request() (like search and all that stuff) :)

All is good to go. My next plan is to clean up internal functions to acutally use self._request() (like search and all that stuff) :)
ryanmcgrath commented 2012-04-08 17:32:59 -07:00 (Migrated from github.com)

Mmm, I know this is probably a bit of a hassle, but can you update this to be in-line with the other recent pull requests? Feel free to merge in your self._request() bit as well if you have it, sure it would rock. ;)

Mmm, I know this is _probably_ a bit of a hassle, but can you update this to be in-line with the other recent pull requests? Feel free to merge in your `self._request()` bit as well if you have it, sure it would rock. ;)
michaelhelmick commented 2012-04-08 17:53:02 -07:00 (Migrated from github.com)

There is a problem that this can't be merged in automatically?

Sent from my iPhone

On Apr 8, 2012, at 8:33 PM, Ryan McGrathreply@reply.github.com wrote:

Mmm, I know this is probably a bit of a hassle, but can you update this to be in-line with the other recent pull requests? Feel free to merge in your self._request() bit as well if you have it, sure it would rock. ;)


Reply to this email directly or view it on GitHub:
https://github.com/ryanmcgrath/twython/pull/72#issuecomment-5019600

There is a problem that this can't be merged in automatically? Sent from my iPhone On Apr 8, 2012, at 8:33 PM, Ryan McGrathreply@reply.github.com wrote: > Mmm, I know this is _probably_ a bit of a hassle, but can you update this to be in-line with the other recent pull requests? Feel free to merge in your `self._request()` bit as well if you have it, sure it would rock. ;) > > --- > > Reply to this email directly or view it on GitHub: > https://github.com/ryanmcgrath/twython/pull/72#issuecomment-5019600
michaelhelmick commented 2012-04-08 17:56:14 -07:00 (Migrated from github.com)

Not sure what would have changed with the oauth callback pull request and my additions? I didn't change anything in any of the code he edited

Sent from my iPhone

On Apr 8, 2012, at 8:33 PM, Ryan McGrathreply@reply.github.com wrote:

Mmm, I know this is probably a bit of a hassle, but can you update this to be in-line with the other recent pull requests? Feel free to merge in your self._request() bit as well if you have it, sure it would rock. ;)


Reply to this email directly or view it on GitHub:
https://github.com/ryanmcgrath/twython/pull/72#issuecomment-5019600

Not sure what would have changed with the oauth callback pull request and my additions? I didn't change anything in any of the code he edited Sent from my iPhone On Apr 8, 2012, at 8:33 PM, Ryan McGrathreply@reply.github.com wrote: > Mmm, I know this is _probably_ a bit of a hassle, but can you update this to be in-line with the other recent pull requests? Feel free to merge in your `self._request()` bit as well if you have it, sure it would rock. ;) > > --- > > Reply to this email directly or view it on GitHub: > https://github.com/ryanmcgrath/twython/pull/72#issuecomment-5019600
michaelhelmick commented 2012-04-09 07:59:34 -07:00 (Migrated from github.com)

Check it out. :) Hopefully 1.6.0 can be pushed out tonight? :)

Check it out. :) Hopefully 1.6.0 can be pushed out tonight? :)
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: code/twython#72
No description provided.