← Back to team overview

banking-addons-team team mailing list archive

Re: banking_addons: Upgrade to 6.1

 

On 03/08/2012 01:22 PM, James Jesudason wrote:


On 08/03/12 10:32, Stefan Rijnhart wrote:
On 03/07/2012 06:06 PM, James Jesudason wrote:
One irritating error that I can't seem to track down is this:
 * Uncaught Error: Unknown field state in domain [["state","not
in",["draft"]]]
This happens when a statement line is clicked in the bank statement
form. Any ideas?

Had a look with Firebug just now and found out that the web client now
checks the domain for the readonly attribute from the 'states' parameter
in the 'amount' column definition even when the whole form is readonly.
Knowing that it is easily solved by injecting the 'state' field into the
embedded form on the bank statement form view. I prepared a branch with
this fix and another fix for the online bank account lookup that is used
in the Netherlands.
Great - could you create a merge proposal for this?


If you have not got the invitation for review yet, see here: https://code.launchpad.net/~therp-nl/banking-addons/6.1-dev-fix_code_and_state/+merge/96539

The upgrade has focused on the functionality that I'm familiar with and can test. Since the module handles a lot more scenarios than we use, I cannot test all of them. So, I think it is better if I could ask others to start testing this branch and whether we could move to a new phase of the upgrade.

Thank you for the clear summary. Can you give me a couple of days to do some testing myself?

Regards,
Stefan.

--
Therp - Maatwerk in open ontwikkeling

Stefan Rijnhart - Ontwerp en implementatie

mail: stefan@xxxxxxxx
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com/therp_stefan



Follow ups

References