Code review comment for lp:~openerp-dev/openobject-addons/trunk-bug-791087-mtr

Revision history for this message
Olivier Dony (Odoo) (odo-openerp) wrote :

Hello,

I don't think removing the demo data records is the good solution, as it is interesting to have entries in the daily attendances when you install a new demo database.

Couldn't we instead replace the XML demo data with a YML demo file that does a smarter job: create 8 hours of attendance for every day of the current business week, using proper timedelta calculations, and making no conflict?

We could also do the same for the analytic entries, to have a match between them.

Thanks!

(I'm resetting the bug to Confirmed, as I don't think this is a proper fix)

review: Disapprove

« Back to merge proposal