Merge lp://qastaging/~anybox/wms/burst-cancelled into lp://qastaging/wms

Proposed by Georges Racinet
Status: Needs review
Proposed branch: lp://qastaging/~anybox/wms/burst-cancelled
Merge into: lp://qastaging/wms
Diff against target: 135 lines (+103/-4)
3 files modified
wms/__terp__.py (+2/-1)
wms/demo/burst_out_cancelled_line.xml (+85/-0)
wms/object/burst.py (+16/-3)
To merge this branch: bzr merge lp://qastaging/~anybox/wms/burst-cancelled
Reviewer Review Type Date Requested Status
Christophe CHAUVET Needs Fixing
Review via email: mp+71049@code.qastaging.launchpad.net

Description of the change

See the linked bug.

To post a comment you must log in.
Revision history for this message
Christophe CHAUVET (christophe-chauvet) wrote :

Hi Georges

Can you replace the browse by a read ?

Thanks,

Regards,

review: Needs Fixing
Revision history for this message
Georges Racinet (gracinet) wrote :

> Can you replace the browse by a read ?

Sorry, forgot to extract that as well (done).

Revision history for this message
Christophe CHAUVET (christophe-chauvet) wrote :

Georges

Have you push the last version for your branch ?

Regards,

Unmerged revisions

231. By Georges Racinet <gracinet@nlg>

[FIX] check state of move lines and special case for the cancelled ones.

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