[12.0][MIGR] product_supplierinfo_for_customer #446

renatonlima · 2019-04-16T23:47:18Z

No description provided.

…option

…elds

…lder.

…tomer: Complete README.

…ext key is needed in act_window Or the product template won't be shown there.

…ling The same way Odoo does.

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: product-attribute-11.0/product-attribute-11.0-product_supplierinfo_for_customer Translate-URL: https://translation.odoo-community.org/projects/product-attribute-11-0/product-attribute-11-0-product_supplierinfo_for_customer/

pedrobaeza · 2019-04-17T00:09:37Z

With this module, I wanted to change a bit the implementation, doing an inheritance of product.supplierinfo, but changing the name, and thus, we avoid the hack in search method plus the modification of supplier info views. What do you think about this change? I can do it if you prefer.

Check also the red status.

…convert instead

renatonlima · 2019-04-17T18:09:04Z

@pedrobaeza,

I will push some fixes to red status for this PR. About your changes in this module, I agree, maybe we can create one object inherited from product.supplierinfo with name product.customerinfo and could we rename this module to product_customerinfo? What do you thing? I can do this :)

pedrobaeza · 2019-04-17T19:36:53Z

I don't see the need of renaming the module, as this denotes the source of the data, but thanks for doing the refactoring.

youring · 2019-05-27T03:21:43Z

product_supplierinfo_for_customer/models/product_product.py

+                    continue
+                prices[product.id] = price
+                if not uom and self._context.get('uom'):
+                    uom = self.env['product.uom'].browse(self._context['uom'])


Replace product.uom with uom.uom, price list based on partner functional test ok

@renatonlima please

rafaelbn · 2019-09-04T23:11:50Z

Dear @renatonlima could you please re-take this PR a little bit? You have comments from @youring and @mreficent and @pedrobaeza weill give last feedback, me too.

Also @murtuzasaleh could review this PR and help as is interested for continuing with his PR OCA/sale-workflow#841 😄

OCA force! Let's go!

pedrobaeza · 2019-09-04T23:21:57Z

Yes, I'm not sure if @renatonlima understood what I wanted to do. Basically it's:

class ProductCustomerinfo(models.Model):
    _inherit = "product.supplierinfo"
    _name = "product.customerinfo"

class ProductTemplate(models.Model):
    _inherit = "product.template"

    customer_ids = fields.One2many(
        comodel_name="product.customerinfo"
        ...

and the rest of the needed stuff.

MiquelRForgeFlow · 2019-10-24T15:49:01Z

Superseded in #515.

oihane and others added 30 commits April 16, 2019 18:54

<product_supplierinfo_for_customer>

c4979bb

product_supplierinfo_for_customer: Views and documentation

7ac447a

Related to OCA#638

82e1a9d

Slovene translations of odoomrp modules

0612b5b

product_supplierinfo_for_customer: Avoid side effects + improve README

172d3db

<product_supplierinfo_for_customer> Create 'customer' type from menu …

14f5c9c

…option

<product_supplierinfo_for_customer> Added know issue with computed fi…

3e60268

…elds

product_supplierinfo_for_customer: Create "demo" data, and "tests" fo…

87f7246

…lder.

license target link

cb9c7e0

OCA as author added

6dfcbb6

OCA Transbot updated translations from Transifex

39f86a9

product_customer_code: New README. [REF] product_supplierinfo_for_cus…

cc84280

…tomer: Complete README.

product_supplierinfo_for_customer to v9

368b5c0

[MIG] product_supplierinfo_for_customer to v10

6e016dc

[IMP] Add back the option to import the prices from supplierinfo to PL

deb6c8d

[UPD] Update product_supplierinfo_for_customer.pot

d9ecd7a

[FIX] Move some views to product_supplierinfo_for_customer_sale

b00bd6f

[UPD] Update product_supplierinfo_for_customer.pot

d26fac3

[REF] Don't overwrite views and fix view for customers

437aa0c

[UPD] Update product_supplierinfo_for_customer.pot

0bc32ea

fix view field names

2b60c58

[UPD] Update product_supplierinfo_for_customer.pot

579e284

[IMP] Readem title and optimization

01634b6

[FIX] Assure partner is passed in the context

fbb850b

[11.0][MIG] product_supplier_info_for_customer: Module migration

eef8437

[IMP] Optimization and fixes from reviews

8f7feec

[IMP] Use variant one2many introduced in v11 like variant_seller_ids

a0f6488

[IMP] product.supplierinfo: type renamed to supplierinfo_type

1976811

[IMP] Change to SavepointCase some tests

30d30c7

[UPD] README.rst

d398a2d

pedrobaeza and others added 7 commits April 16, 2019 18:54

[FIX] product_supplierinfo_for_customer: visible_product_tmpl_id cont…

b303d69

…ext key is needed in act_window Or the product template won't be shown there.

[FIX] product_supplierinfo_for_customer: Adapt views to variants hand…

f2c42d8

…ling The same way Odoo does.

[UPD] Update product_supplierinfo_for_customer.pot

f4a5d43

[12.0][MIGR] __manifest__

dc1c154

[12.0][MIGR] product.supplierinfo object

09f6530

[12.0][MIGR] product.template object

ad40e6a

pedrobaeza added this to the 12.0 milestone Apr 17, 2019

renatonlima added 2 commits April 17, 2019 10:13

[12.0][MIGR] product and product.supplierinfo views

3c67912

[12.0][MIGR] replace deprecated compute method in res.currency, use _…

bb42340

…convert instead

renatonlima force-pushed the 12.0-mig-product_supplierinfo_for_customer branch from 1cfef43 to bb42340 Compare April 17, 2019 14:52

renatonlima changed the title ~~[12.0] product_supplierinfo_for_customer migration~~ [12.0][MIGR] product_supplierinfo_for_customer migration Apr 17, 2019

renatonlima changed the title ~~[12.0][MIGR] product_supplierinfo_for_customer migration~~ [12.0][MIGR] product_supplierinfo_for_customer Apr 17, 2019

renatonlima added 2 commits April 17, 2019 19:32

[12.0][MIGR] Module tests

1de80cc

pep8

343b061

renatonlima force-pushed the 12.0-mig-product_supplierinfo_for_customer branch from 5505fe4 to 343b061 Compare April 17, 2019 22:34

murtuzasaleh mentioned this pull request May 8, 2019

[12.0][MIG] product_supplierinfo_for_customer_sale OCA/sale-workflow#841

Closed

youring reviewed May 27, 2019

View reviewed changes

emagdalenaC2i mentioned this pull request Jun 1, 2019

Migration to version 12.0 #389

Closed

24 tasks

This was referenced Sep 4, 2019

[8.0] product_supplierinfo_for_customer_sale: Description #335

Closed

[12.0] product_supplierinfo_for_customer #454

Closed

[product_supplierinfo_for_customer_sale] Change functionality from v8 to v10 to v11 OCA/sale-workflow#737

Closed

MiquelRForgeFlow mentioned this pull request Oct 15, 2019

[12.0][MIG] product_supplierinfo_for_customer #515

Merged

pedrobaeza closed this Oct 24, 2019

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[12.0][MIGR] product_supplierinfo_for_customer #446

[12.0][MIGR] product_supplierinfo_for_customer #446

renatonlima commented Apr 16, 2019

pedrobaeza commented Apr 17, 2019

renatonlima commented Apr 17, 2019

pedrobaeza commented Apr 17, 2019

youring May 27, 2019

MiquelRForgeFlow Aug 2, 2019

rafaelbn commented Sep 4, 2019

pedrobaeza commented Sep 4, 2019

MiquelRForgeFlow commented Oct 24, 2019

[12.0][MIGR] product_supplierinfo_for_customer #446

[12.0][MIGR] product_supplierinfo_for_customer #446

Conversation

renatonlima commented Apr 16, 2019

pedrobaeza commented Apr 17, 2019

renatonlima commented Apr 17, 2019

pedrobaeza commented Apr 17, 2019

youring May 27, 2019

Choose a reason for hiding this comment

MiquelRForgeFlow Aug 2, 2019

Choose a reason for hiding this comment

rafaelbn commented Sep 4, 2019

pedrobaeza commented Sep 4, 2019

MiquelRForgeFlow commented Oct 24, 2019