openerp-community-reviewer team mailing list archive

["Luc De Meyer \(Noviat\)" <luc.demeyer@xxxxxxxxx>]

Period is a char field to facilitate the search on all periods in a given fiscal year (assuming of course that the fiscal year name is consistently used in the naming of the periods). 

Luc


www.noviat.com
Rusatiralaan 1, 1083 Brussel
+32 2 808 86 38



-----Original Message-----
From: bounces@xxxxxxxxxxxxx [mailto:bounces@xxxxxxxxxxxxx] On Behalf Of Guewen Baconnier @ Camptocamp
Sent: vrijdag 21 maart 2014 9:18
To: mp+211967@xxxxxxxxxxxxxxxxxx
Subject: 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
You are the owner of lp:~luc-demeyer/account-financial-tools/add-account_move_line_search_extension.


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.