Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature Request: username selection command line option #10 #105

Closed

Conversation

mattjbones
Copy link

Problem

As detailed in #10:

The two step process to ensure the correct username is used (in a script for example):

twurl set default username
twurl tweet -d status="why's that?"

Seems a bit awkward. Couldn't -u be used for this?

Solution

Update oauth_client.rb to handle-u flag for selecting username (as explained in #10)

Result

The following:

$ twurl -u user_one /1.1/statuses/home_timeline.json
$ twurl -u user_two /1.1/statuses/home_timeline.json

should yield different results

@CLAassistant
Copy link

CLAassistant commented Oct 20, 2018

CLA assistant check
All committers have signed the CLA.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 88.004% when pulling b7f1f8a on mattjbones:username-selection-by-flag into 2f84302 on twitter:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 88.004% when pulling b7f1f8a on mattjbones:username-selection-by-flag into 2f84302 on twitter:master.

@coveralls
Copy link

coveralls commented Oct 20, 2018

Coverage Status

Coverage decreased (-0.08%) to 88.085% when pulling b7f1f8a on mattjbones:username-selection-by-flag into 2f84302 on twitter:master.

@mattjbones mattjbones closed this Oct 20, 2018
@andypiper
Copy link
Contributor

Did you intend to close this without having it merged?

@mattjbones
Copy link
Author

mattjbones commented Oct 21, 2018 via email

@AgoraSecurity
Copy link

Any news on this? I'ld love to have this feature. I've been working on how to make this work for some days now

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

Successfully merging this pull request may close these issues.

5 participants