Skip to content

Make postTweet response include errors#183

Open
johnbailon wants to merge 1 commit intosivchari:mainfrom
johnbailon:post-tweet-response-error
Open

Make postTweet response include errors#183
johnbailon wants to merge 1 commit intosivchari:mainfrom
johnbailon:post-tweet-response-error

Conversation

@johnbailon
Copy link
Copy Markdown

Post tweet doesn't return the full response when the request is not successful. I added details from Twitter API's response to the returned object.

@sivchari
Copy link
Copy Markdown
Owner

HI, @johnbailon
Looks good to me. Just in case, please give me the documentation about this. Thanks.

@johnbailon
Copy link
Copy Markdown
Author

johnbailon commented Sep 29, 2023

HI, @johnbailon
Looks good to me. Just in case, please give me the documentation about this. Thanks.

Sure @sivchari, basically, I just added the error that would come from Twitter on to the response of PostTweet. I made sure to follow your conventions and I modified the test to take into account the changes I made.

Context: I needed to know what was causing my PostTweet requests to fail and the current version doesn't provide the erro response from Twitter.

@sivchari
Copy link
Copy Markdown
Owner

@johnbailon
Sorry, too late to respond. I concern if this response is really had errors field. Have you confirmed it ? If yes, please paste the response. Because I want to confirm it just in case.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants