Banking Addons

Code review comment for lp:~ruchir.shukla/banking-addons/banking-addons-payment-order-improvment

  1. banking-addons-payment-order-improvment
  2. Merge into banking-addons-70
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Hi Ruchir,

sorry I missed this one. Please check if the code needs updating when you activate a module! In this case, it is thankfully only a very small number of changes. Additionally, I think reloading the payment order view is a workaround for https://bugs.launchpad.net/openerp-web/+bug/1155525 so this had better be pointed out in a comment. You can find these small changes here: https://code.launchpad.net/~therp-nl/banking-addons/ba70-fix_payment_order_improvements/+merge/204485.

Could you review my additional changes to this proposal?

Thanks!

« Back to merge proposal