Page MenuHomeDevCentral

Set Twitter message length
ClosedPublic

Authored by dereckson on Oct 12 2017, 16:12.
Tags
None
Referenced Files
F3763100: D1048.diff
Thu, Nov 21, 20:32
Unknown Object (File)
Tue, Nov 12, 23:52
Unknown Object (File)
Tue, Nov 12, 17:24
Unknown Object (File)
Tue, Nov 12, 11:41
Unknown Object (File)
Thu, Nov 7, 19:31
Unknown Object (File)
Tue, Nov 5, 10:43
Unknown Object (File)
Tue, Nov 5, 09:32
Unknown Object (File)
Thu, Oct 31, 12:45
Subscribers
None

Details

Summary

This change prepares for https://www.theverge.com/2017/9/26/16363912/twitter-character-limit-increase-280-test

Currently, the API refuses statuses > 140 characters, so we can't set it to 280.

Ref T1249.

Test Plan

Fire a longer message than 140 characters.

Diff Detail

Repository
rVIPER ViperServ scripts
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Oct 13 2017, 22:34

Get rid of the global variable, so we can use .s to load it.

Get rid of the global variable.

This revision is now accepted and ready to land.Oct 13 2017, 22:41

The 280 limit is for web only:

2:40:41 <Daeghrefn> [DEBUG] Twitter error: OAuth failure: (code: 403) {"errors":[{"code":186,"message":"Status is over 140 characters."}]}
22:41:07 <Dereckson> .tcl putdebug
22:41:07 <Daeghrefn> [DEBUG] OAuth failure: (code: 403) {"errors":[{"code":186,"message":"Status is over 140 characters."}]}
22:41:07 <Daeghrefn>     while executing
22:41:07 <Daeghrefn> "error "OAuth failure: (code: $ncode) $data""
22:41:07 <Daeghrefn>     (procedure "oauth::query" line 16)
22:41:07 <Daeghrefn>     invoked from within
22:41:07 <Daeghrefn> "oauth::query $url $method $oauth_header $oauth_query"
22:41:07 <Daeghrefn>     (procedure "oauth::query_call" line 25)
22:41:07 <Daeghrefn>     invoked from within
22:41:07 <Daeghrefn> "oauth::query_call $url $consumer_key $consumer_secret $method $params $query_dict $oauth_token_secret"
22:41:07 <Daeghrefn>     (procedure "oauth::query_api" line 3)
22:41:07 <Daeghrefn>     invoked from within
22:41:07 <Daeghrefn> "oauth::query_api $url {*}[registry get twitter.oauth.consumer] $method {*}$token $query_list"
22:41:07 <Daeghrefn>     (procedure "twitter_query" line 20)
22:41:07 <Daeghrefn>     invoked from within
22:41:07 <Daeghrefn> "twitter_query $status_url $account [list status $message]"
This revision now requires review to proceed.Oct 14 2017, 00:07

We can land this with 140 as current setting actually.

dereckson retitled this revision from Set Twitter message length to 280 to Set Twitter message length.Oct 14 2017, 22:56
dereckson edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Oct 14 2017, 23:01
This revision was automatically updated to reflect the committed changes.