launchpad-dev team mailing list archive

[Robert Collins <robert.collins@xxxxxxxxxxxxx>]

On Wed, Sep 29, 2010 at 8:19 PM, Stuart Bishop
<stuart.bishop@xxxxxxxxxxxxx> wrote:
>>> And if get_current_browser_request always works, can we put it in
>>> get_request_timeline instead of typing it explicitly every time?
>>
>> So there are two disconnected steps:
>>  - get a request from some context
>>  - get the timeline from that request (which will use IAnnotations
>> once bug 623199 is fixed)
>>
>> I'm not too keen on conflating these steps; we could certainly add a
>> new helper get_current_browser_request_timeline() though, that would
>> be fine.
>
> For scripts, consider adding this as an attribute on LaunchpadScript
> so everyone gets to play.

hmm? scripts can use features already; fixing bug 623199 would make
that be better structured, but theres no functional issue today.

-Rob