Code review comment for lp:~sbi/account-financial-report/account-financial-report-v7bug20130410

Revision history for this message
Stéphane Bidoul (Acsone) (sbi) wrote :

Hi,

The purpose of revno 26 was to fix the views for 7.0 (ie fix the display of
last_rec_date).
Why do you say it's no longer displayed?

Anyway, we don't use last_rec_date so if you believe it's not necessary,
you can drop it from the views.

-sbi

On Tue, May 7, 2013 at 10:53 AM, Guewen Baconnier @ Camptocamp <
<email address hidden>> wrote:

> Hi,
>
> I was about to merge revno 26 / revid.
> <email address hidden>
> from lp:account-financial-report/trunk to lp:account-financial-report/7.0
> when I noticed that the last_rec_date field is no longer displayed at all
> in the views in the 7.0 version.
>
> I asked our functional guy who thinks that the field is not worth to be
> displayed and does not bring useful information for the users.
>
> Do you agree or do you see a reason to keep the field in the views?
>
> Thanks
> --
>
> https://code.launchpad.net/~sbi/account-financial-report/account-financial-report-v7bug20130410/+merge/158031
> You are the owner of
> lp:~sbi/account-financial-report/account-financial-report-v7bug20130410.
>

« Back to merge proposal