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

Optional: Detect shortened URLs and replace with longer #11

Closed
naholyr opened this issue Mar 27, 2011 · 7 comments
Closed

Optional: Detect shortened URLs and replace with longer #11

naholyr opened this issue Mar 27, 2011 · 7 comments

Comments

@naholyr
Copy link

naholyr commented Mar 27, 2011

Should be optional, and may be enabled with a command such like ":enable auto_expand_urls".
Then messages will appear with shortened URLs replaced with the full ones.

Could be called manually with a command like ":expand_urls " (prints the message with full URLs) and/or ":expand_url " (prints full url).

@ip2k
Copy link
Contributor

ip2k commented Aug 31, 2011

IMO this matters more now that Twitter has integrated the t.co stuff: https://dev.twitter.com/discussions/1062

@indeyets
Copy link

indeyets commented Mar 5, 2012

+1 this is a standard option of many other clients

@seppo0010
Copy link
Contributor

I think this is already implemented while streaming tweets, but not when using :recent.

@no6v
Copy link
Collaborator

no6v commented Mar 6, 2012

Appending "include_entities=true" to GET request,
we can use entities for url, media, user_mention and hashtag
even when using :recent, :status, :search and so on.
Coloring for hashtag may be also improved if entity is always available.

@jugyo
Copy link
Owner

jugyo commented Mar 6, 2012

Sounds good! Do you have the patch?

@no6v
Copy link
Collaborator

no6v commented Mar 6, 2012

https://gist.github.com/1651877 is a draft patch for this purpose.
Appending parameter to request is the bottom part of the patch.
I feel this is a bit overkill and yet incomplete. I'd love to brush it up.

@no6v
Copy link
Collaborator

no6v commented Mar 8, 2012

Pull-requested as #108. Let's start a discussion over there.

@no6v no6v closed this as completed Mar 8, 2012
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

No branches or pull requests

6 participants