[8.0][MIG] partner_identification backport #612
Conversation
pankk
force-pushed
the
8.0-partner-identification-backport
branch
from
6c8da51 to
4c12b3a
Compare
| 'views/res_partner_id_category_view.xml', | ||
| 'views/res_partner_id_number_view.xml', | ||
| 'views/res_partner_view.xml', | ||
| 'security/ir.model.access.csv', | ||
| ], | ||
| 'author': 'ChriCar Beteiligungs- und Beratungs- GmbH, ' | ||
| 'Antiun Ingeniería S.L.', |
There was a problem hiding this comment.
Can you move the comma inside the string
There was a problem hiding this comment.
Thanks @tarteo, I will cherry-pick the commits from 11.0 that solve this (and some other) issues.
There was a problem hiding this comment.
Can you please update Antiun author to the new company Tecnativa?
There was a problem hiding this comment.
@pedrobaeza Yes, I've seen commits that push that change and will cherry-pick them. Thanks!
| Returns: | ||
| list: Domain to search with. | ||
| """ | ||
| id_numbers = self.env['res.partner.id_number'].search([ |
There was a problem hiding this comment.
@pankk Avoid this (I know this is on v11 - I'll try to fix it later).
It's not necessary to do a search before returning (two queries instead of one).
return [
('id_numbers.name', operator, value),
('id_numbers.category_id.code', '=', category_code),
]
It will impact performances.
There was a problem hiding this comment.
I get #434 when doing it like this. Can we go with @pedrobaeza's suggestion in #436?
There was a problem hiding this comment.
@pankk Yes, I was not aware of that bug. Ok for me
pankk
force-pushed
the
8.0-partner-identification-backport
branch
3 times, most recently
from
f401813 to
3699e53
Compare
(cherry picked from commit eadcae2) Conflicts: partner_identification/__openerp__.py Set version to 8.0.1.0.0. Remove dependency on sales_team, as the relevant change is not in 8.0. Change emails to the new ones (cherry picked from commit 3455ae6) Update new name (cherry picked from commit 5b592d7) [FIX] try me on runbot link (cherry picked from commit bd587b6) Conflicts: partner_identification/README.rst Changed runbot to 8.0 [IMP] partner_identification: Add context override (OCA#373) Allow for context override of validations using ``id_no_validate`` (cherry picked from commit 76c2e7b) [10.0][IMP] partner_identification: Add field computation and inverses (OCA#419) * [IMP] partner_identification: Add field computation and inverses * Add methods to allow for computation and inverse of an ID field of a specific category type * [IMP] partner_identification: Add search option (cherry picked from commit 19c5fb6) [FIX] partner_identification: Infinite loop in search (OCA#436) (cherry picked from commit fa9b390) [FIX] partner-contact CI interactions (cherry picked from commit bc93e7b) [ADD][8.0] Backport of the 9.0 module. (cherry picked from commit a425403) [8.0][MIG] partner_identification backport
pankk
force-pushed
the
8.0-partner-identification-backport
branch
from
3699e53 to
3e7004b
Compare
|
Merging as the lint error is unrelated |
Backport of partner_identification, based on the 10.0 version which adds additional features. These features are required for the partner_coc PR here: #605