Code review comment for lp:~openerp-dev/openobject-addons/7.0-base-contact-xal

Revision history for this message
Pierre Lamarche (www.savoirfairelinux.com) (pierre-lamarche) wrote :

Hello,

First of all, thanks for this work.

Here are our review and comments about the functionnalities :

* One contact should have multiple relation with companies on the same level.
(why differenciate a main one and other positions ?)
We should be able to leave blank the parent company field on the top and only add multiple positions in the "Other Positions" tab below

* When you are creating a contact directly on the already opened partner form of a company, it only creates the database entry for the contact, company instead of create a one entry for the contact alone, and one entry for the contact, company.

* The idea is to manage a list of contact, a list of company and a list of relation between them.
one contact can have several relations with one or several companies, a company can have several relations with one or several contacts. Each relation should be based on a job position with professional addresses, telephone, etc...

I would be happy to discuss about that, Xavier.

BR,

Pierre

« Back to merge proposal