Merge lp://staging/~jpds/arsenal/css-subscribed into lp://staging/arsenal/trunk

Proposed by Jonathan Davies
Status: Needs review
Proposed branch: lp://staging/~jpds/arsenal/css-subscribed
Merge into: lp://staging/arsenal/trunk
Diff against target: 159 lines (+92/-5)
3 files modified
reports/cbd (+25/-5)
reports/css/rls-mgr-p-css-tracking.json (+35/-0)
reports/css/rls-p-css-tracking.json (+32/-0)
To merge this branch: bzr merge lp://staging/~jpds/arsenal/css-subscribed
Reviewer Review Type Date Requested Status
Brad Figg code Pending
Review via email: mp+98529@code.staging.launchpad.net

Description of the change

This branch allows one to specify a subscriber field to JSON files. Only bugs with that subscriber will be thus listed.

To post a comment you must log in.
Revision history for this message
Brad Figg (brad-figg) wrote :

On 03/20/2012 03:15 PM, Jonathan Davies wrote:
> Jonathan Davies has proposed merging lp:~jpds/arsenal/css-subscribed into lp:arsenal.
>
> Requested reviews:
> Brad Figg (brad-figg): code
>
> For more details, see:
> https://code.launchpad.net/~jpds/arsenal/css-subscribed/+merge/98529
>
> This branch allows one to specify a subscriber field to JSON files. Only bugs with that subscriber will be thus listed.

This fails for .json files that do not have a "subscriber" entry
in them. That will need to be fixed.

Brad
--
Brad Figg <email address hidden> http://www.canonical.com

963. By Jonathan Davies

Make reports/cbd set search_subscriber to None instead of an empty string.

Revision history for this message
Jonathan Davies (jpds) wrote :

> This fails for .json files that do not have a "subscriber" entry
> in them. That will need to be fixed.

Fixed in r963.

964. By Jonathan Davies

Finalize CSS bug reports.

Unmerged revisions

964. By Jonathan Davies

Finalize CSS bug reports.

963. By Jonathan Davies

Make reports/cbd set search_subscriber to None instead of an empty string.

962. By Jonathan Davies

Fixed Dbg output for subscriber.

961. By Jonathan Davies

Improved search_subscriber code.

960. By Jonathan Davies

Corrected subscribers → subscriber typo change.

959. By Jonathan Davies

Added reports/rls-mgr/rls-mgr-p-css-tracking.json.

958. By Jonathan Davies

Adapted cbd to take a subscriber option.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: