Thanks Virgil, Stefan. I know it is hard work to review code...
In case this helps reviewers, this change is already applied to 7.0 and trunk branches.
I just checked that underline modules had not changed between versions and then applied the same changes...
If there is anything I can do to help, just let me know. El 20/09/2013 06:31, "Virgil Dupras" <email address hidden> escribió:
> I just want to say that I'm not ignoring this MP. I will review it, but > being unfamiliar with the module and problem domain (cost of goods sold, > anglo-saxon vs continental, etc.), I have a lot of catching on to do to be > able to do a meaningful review. > -- > > https://code.launchpad.net/~the-clone-master/ocb-addons/6.1-lp212930-purchase_anglo_saxon_invoice_from_PO_lines/+merge/180621 > You are the owner of > lp:~the-clone-master/ocb-addons/6.1-lp212930-purchase_anglo_saxon_invoice_from_PO_lines. >
« Back to merge proposal
Thanks Virgil, Stefan. I know it is hard work to review code...
In case this helps reviewers, this change is already applied to 7.0 and
trunk branches.
I just checked that underline modules had not changed between versions and
then applied the same changes...
If there is anything I can do to help, just let me know.
El 20/09/2013 06:31, "Virgil Dupras" <email address hidden>
escribió:
> I just want to say that I'm not ignoring this MP. I will review it, but /code.launchpad .net/~the- clone-master/ ocb-addons/ 6.1-lp212930- purchase_ anglo_saxon_ invoice_ from_PO_ lines/+ merge/180621
> being unfamiliar with the module and problem domain (cost of goods sold,
> anglo-saxon vs continental, etc.), I have a lot of catching on to do to be
> able to do a meaningful review.
> --
>
> https:/
> You are the owner of
> lp:~the-clone-master/ocb-addons/6.1-lp212930-purchase_anglo_saxon_invoice_from_PO_lines.
>