Page MenuHomeDevCentral

Set Twitter message length
ClosedPublic

Authored by dereckson on Oct 12 2017, 16:12.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 22, 03:58
Unknown Object (File)
Fri, Nov 22, 03:21
Unknown Object (File)
Fri, Nov 22, 02:21
Unknown Object (File)
Fri, Nov 22, 01:03
Unknown Object (File)
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
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
No Lint Coverage
Unit
No Test Coverage
Branch
arcpatch-D1048
Build Status
Buildable 1661
Build 1909: arc lint + arc unit

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.