← Back to team overview

openerp-community-reviewer team mailing list archive

Re: [Merge] lp:~luc-demeyer/account-financial-tools/add-account_move_line_search_extension into lp:account-financial-tools

 

Review: Needs Information code review, test

Hi,

Sounds good. It would be so nice to have autocompletion on the fields (but could be a new feature to add later, that's good enough now). I'm just wondering why the 'period' field is a char field and not a selection field like in the standard view, is it a usability reason?

Thanks
-- 
https://code.launchpad.net/~luc-demeyer/account-financial-tools/add-account_move_line_search_extension/+merge/211967
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:account-financial-tools.


Follow ups

References