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

Adds --full-text option to direct-messages command. Fixes #371 #373

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

porras
Copy link

@porras porras commented Jul 7, 2017

What title says. This option is supported directly by the twitter gem and API so it's a very small change.

    bundle exec bin/t help direct_messages
    Usage:
      t direct_messages

    Options:
      -c, [--csv], [--no-csv]                        # Output in CSV format.
      -d, [--decode-uris], [--no-decode-uris]        # Decodes t.co URLs into their original form.
      -l, [--long], [--no-long]                      # Output in long format.
=>    -f, [--full-text], [--no-full-text]            # Fetch full texts.                <=
      -n, [--number=N]                               # Limit the number of results.
                                                     # Default: 20
      -a, [--relative-dates], [--no-relative-dates]  # Show relative dates.
      -r, [--reverse], [--no-reverse]                # Reverse the order of the sort.
      -C, [--color=COLOR]                            # Control how color is used in output
                                                     # Default: auto
                                                     # Possible values: icon, auto, never
      -P, [--profile=FILE]                           # Path to RC file
                                                     # Default: /Users/sergio/.trc

    Returns the 20 most recent Direct Messages sent to you.

I guess it would make sense to make this the default when --csv is present too, but I didn't want to change the behaviour of a preexisting option unless maintainers suggest so (feel free to suggest this or any other change).

Thanks for the t tool, it's awesome!

@sferik sferik force-pushed the master branch 4 times, most recently from 569eee2 to 00389d0 Compare November 18, 2017 22:50
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.

None yet

1 participant