Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[13.0][MIG] product_multi_ean #259

Closed

Conversation

jimhoefnagels
Copy link

No description provided.

@jimhoefnagels jimhoefnagels force-pushed the 13.0-mig-product_multi_ean branch 4 times, most recently from 1b3c574 to fe84412 Compare April 7, 2020 08:14
@rousseldenis rousseldenis added this to the 13.0 milestone Apr 7, 2020
@jimhoefnagels jimhoefnagels mentioned this pull request Apr 9, 2020
9 tasks
@jimhoefnagels jimhoefnagels changed the title 13.0 mig product multi ean [13.0][MIG] product multi ean Apr 9, 2020
@jimhoefnagels jimhoefnagels changed the title [13.0][MIG] product multi ean [13.0][MIG] product_multi_ean Apr 9, 2020
nbessi and others added 19 commits July 30, 2020 11:39
(lp:c2c-addons/6.1  rev 28.1.10)
[IMP] - the browse method will keep the context in write and create methods of barcode osv
[RVW] - Code review for the v7.0
-> replace osv.osv by orm.Model (and osv.osv_memory by orm.TransientModel)
-> modules declarations ("update_xml" and "init_xml", merge into "data", "demo_xml" replace by "demo").
-> increase modules version which have be migrated.
-> in methods, when there is the context which is given replace the "context" by "context=context" to avoid errors
* Tests
* Spanish translation
* Improve label
* Text search + order
Currently translated at 100.0% (13 of 13 strings)

Translation: stock-logistics-barcode-12.0/stock-logistics-barcode-12.0-product_multi_ean
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-barcode-12-0/stock-logistics-barcode-12-0-product_multi_ean/pt_BR/

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: stock-logistics-barcode-12.0/stock-logistics-barcode-12.0-product_multi_ean
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-barcode-12-0/stock-logistics-barcode-12-0-product_multi_ean/

Translated using Weblate (Portuguese (Brazil))

Currently translated at 100.0% (15 of 15 strings)

Translation: stock-logistics-barcode-12.0/stock-logistics-barcode-12.0-product_multi_ean
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-barcode-12-0/stock-logistics-barcode-12-0-product_multi_ean/pt_BR/
Currently translated at 100.0% (15 of 15 strings)

Translation: stock-logistics-barcode-12.0/stock-logistics-barcode-12.0-product_multi_ean
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-barcode-12-0/stock-logistics-barcode-12-0-product_multi_ean/zh_CN/
* avoid conflict with testing a database with stock installed
Copy link

@qgroulard qgroulard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code review 👍

@tonygalmiche
Copy link
Sponsor

I have this error modifying the Barcode field :

image

@jjscarafia
Copy link
Contributor

I've made new PR for this with some improvements #341

@rousseldenis
Copy link
Sponsor Contributor

closing as this has been done there #345

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.