Merge lp:~eh-dem/partner-contact-management/remove_is_company into lp:~partner-contact-core-editors/partner-contact-management/7.0
Proposed by
El Hadji Dem (http://www.savoirfairelinux.com)
Status: | Rejected |
---|---|
Rejected by: | Sandy Carter (http://www.savoirfairelinux.com) |
Proposed branch: | lp:~eh-dem/partner-contact-management/remove_is_company |
Merge into: | lp:~partner-contact-core-editors/partner-contact-management/7.0 |
Diff against target: |
15 lines (+0/-5) 1 file modified
partner_firstname/partner_view.xml (+0/-5) |
To merge this branch: | bzr merge lp:~eh-dem/partner-contact-management/remove_is_company |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Olivier Laurent (Acsone) (community) | Disapprove | ||
Partner and Contact Core Editors | Pending | ||
Review via email: mp+202543@code.launchpad.net |
Description of the change
remove is_company from contact view added in https:/
In partner view, we create a company; from the company, we create a contact and again you add is_company field; it is redundant and the view doesn't display well
To post a comment you must log in.
Unmerged revisions
- 32. By El Hadji Dem (http://www.savoirfairelinux.com)
-
[IMP] remove is_company from contact view added in MP 195092
1/ It is not redundant, it is a new feature allowing a company to be a child of another one. /code.launchpad .net/~nemry/ partner- contact- management/ 7.0-partner- firstname/ +merge/ 202713
2/ The "quality" of the inner form design is solved by https:/