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

14.0 mig product multi ean fix setup #12

Open
wants to merge 2 commits into
base: 14.0-mig-product_multi_ean
Choose a base branch
from

Conversation

michotm
Copy link

@michotm michotm commented Jun 16, 2021

  • Fix setup for pip install
  • Add migration from product_multi_ean

@sebastienbeau
Copy link
Member

Salut ça serait jouable de faire cette PR sur la V13 ? sur cette branches OCA#345
Par contre pour la migraiton il ne faut pas le faire via un pre-init de migration (exemple ici : https://github.com/OCA/account-financial-reporting/tree/14.0/account_tax_balance/migrations/14.0.1.0.0) et pas un pre-init hook

def post_init_hook(cr, registry):
cr.execute("select name,model from ir_model where model = 'product.ean13';")
row = cr.fetchone()
if row:
Copy link
Member

Choose a reason for hiding this comment

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

Cette partie ira de le script de migration (d'ailleur tu peux renommer l'object et les champs) il y a une méthode openupgrade pour ça

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

Successfully merging this pull request may close these issues.

2 participants