Merge lp:~robru/friends-app/140chars into lp:friends-app
Proposed by
Robert Bruce Park
Status: | Merged |
---|---|
Approved by: | Ken VanDine |
Approved revision: | 68 |
Merged at revision: | 70 |
Proposed branch: | lp:~robru/friends-app/140chars |
Merge into: | lp:friends-app |
Diff against target: |
26 lines (+2/-3) 1 file modified
qml/Post.qml (+2/-3) |
To merge this branch: | bzr merge lp:~robru/friends-app/140chars |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot (community) | continuous-integration | Approve | |
Ken VanDine | Approve | ||
Review via email:
|
Commit message
Fix off-by-one error when calculating tweet lengths. (LP: #1168899)
Description of the change
Use <= instead of < to allow 140 char messages.
To post a comment you must log in.
FAILED: Continuous integration, rev:68 /code.launchpad .net/~robru/ friends- app/140chars/ +merge/ 158979/ +edit-commit- message
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https:/
http:// jenkins. qa.ubuntu. com/job/ friends- app-ci/ 1/ jenkins. qa.ubuntu. com/job/ friends- app-raring- amd64-ci/ 1
Executed test runs:
SUCCESS: http://
Click here to trigger a rebuild: s-jenkins: 8080/job/ friends- app-ci/ 1/rebuild
http://