390713 - Trim summaries for newsfeed stories

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Facebook Application, defect)

Description

[Justin Scott [:fligtar]]

Seeing a couple of these...

Aug  2 14:42:37 mrapp51.mozilla.org httpd[2840]: [error] [client 10.2.81.4] _publishStory: Feed story body is too long

Newsfeed story bodies are limited to 200 characters excluding markup.

Comment 1

[Justin Scott [:fligtar]]

Fixed with bug 390678; will be pushed with 0.3.

Comment 2

[Stephen Donner [:stephend] Not actively reading bugmail]

Is this something I/you could verify only by grepping server logs?  I don't know how to control the length of the newsfeed story bodies in order to properly test this...thanks!

Comment 3

[Justin Scott [:fligtar]]

If you add iMacros as a favorite, the summary should be cut off in the minifeed story similar to when browsing/searching.

Comment 4

[Stephen Donner [:stephend] Not actively reading bugmail]

Ok, I can verify that there, we're doing the same thing we do on the browse and search pages:

I got 191 characters (with spaces) for the feed summary on staging; is that cool?

Comment 5

[Justin Scott [:fligtar]]

Yep!

Comment 6

[Stephen Donner [:stephend] Not actively reading bugmail]

OK; I saw your code comments that said you break on clauses (like comments, etc.), so that's great.  Verified FIXED per comment 4, using Firefox 2.0.0.6, Windows XP SP2, Linux (Fedora Core), and Mac OS X 10.4 on staging.