Twitter actually shows a mention if it is preceded with any mixed #58

Merged
merged 1 commit into from Sep 16, 2013

Projects

None yet

2 participants

@kungfumike

case of the characters "r" and "t".
https://twitter.com/sono_inf/statuses/310385896321056769 provides
a lower case example. Extracted streams from Gnip and Datasift
confirm this behavior as well.
This work done on the dime of: Simply Measured simplymeasured.com

@kungfumike kungfumike Twitter actually shows a mention if it is preceded with any mixed
case of the characters "r" and "t".
https://twitter.com/sono_inf/statuses/310385896321056769 provides
a lower case example. Extracted streams from Gnip and Datasift
confirm this behavior as well.
This work done on the dime of: Simply Measured simplymeasured.com
ef0a131
@kungfumike kungfumike referenced this pull request in twitter/twitter-text-rb Sep 13, 2013
Merged

Update Mention regex to include att [Rr][tT] variations #95

@jakl
Member
jakl commented Sep 16, 2013

Good catch, looks great thanks!

@jakl jakl merged commit 4dee4ac into twitter:master Sep 16, 2013
@jimrok jimrok pushed a commit to jimrok/twitter-text-rb that referenced this pull request Jan 20, 2014
@kungfumike kungfumike Update Mention regex to include att [Rr][tT] variations
This commit is actually based on a pull request for the conformance
suite.
twitter/twitter-text-conformance#58
As shown there twitter actually recognizes a mention no matter
what variation of "RT/rt/Rt/rT" preceeds it.
This commit was done on the dime of Simply Measured simplymeasured.com
f992e4f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment