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

[16.0] [ADD] Add helper to migrate translations from ir_translation table to model table #306

Merged
merged 1 commit into from
Sep 26, 2022

Conversation

marielejeune
Copy link
Contributor

No description provided.

@marielejeune marielejeune force-pushed the 16.0-migrate_translations-mle branch from 736cb40 to 084dc5a Compare September 20, 2022 14:54
Copy link
Contributor

@legalsylvain legalsylvain left a comment

Choose a reason for hiding this comment

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

V16 ;-) great.

No review for the time being. I can take a time during the OCA days to review it.

regards.

Copy link

@lmignon lmignon left a comment

Choose a reason for hiding this comment

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

Thank you @marielejeune for this contribution. I propose some changes. See comment.

@pedrobaeza
Copy link
Member

As reference, the Odoo change motivating this PR comes from odoo/odoo#97692

@marielejeune marielejeune force-pushed the 16.0-migrate_translations-mle branch 3 times, most recently from c9985f7 to bc3331a Compare September 21, 2022 08:47
Copy link

@lmignon lmignon left a comment

Choose a reason for hiding this comment

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

Thank you @marielejeune. LGTM (code review, functional test)

Copy link

@adrienpeiffer adrienpeiffer left a comment

Choose a reason for hiding this comment

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

LGTM (Code review)

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@marielejeune marielejeune force-pushed the 16.0-migrate_translations-mle branch 2 times, most recently from 0147b8f to 6b88e48 Compare September 26, 2022 06:59
@lmignon
Copy link

lmignon commented Sep 26, 2022

@pedrobaeza Can you update your review? Thanks!

@pedrobaeza
Copy link
Member

Please fix these flake8 erros:

/home/travis/build/OCA/openupgradelib/build/lib/openupgradelib/openupgrade_160.py:27:80: E501 line too long (80 > 79 characters)
/home/travis/build/OCA/openupgradelib/openupgradelib/openupgrade_160.py:27:80: E501 line too long (80 > 79 characters)

@marielejeune marielejeune force-pushed the 16.0-migrate_translations-mle branch from 6b88e48 to 00ccb79 Compare September 26, 2022 08:55
@marielejeune marielejeune force-pushed the 16.0-migrate_translations-mle branch from 00ccb79 to cf92e08 Compare September 26, 2022 08:56
@marielejeune
Copy link
Contributor Author

@pedrobaeza sorry, done

@pedrobaeza pedrobaeza merged commit dad04c4 into OCA:master Sep 26, 2022
@lmignon lmignon deleted the 16.0-migrate_translations-mle branch September 26, 2022 14:24
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.

7 participants