-
-
Notifications
You must be signed in to change notification settings - Fork 174
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
Conversation
736cb40
to
084dc5a
Compare
There was a problem hiding this 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.
There was a problem hiding this 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.
As reference, the Odoo change motivating this PR comes from odoo/odoo#97692 |
c9985f7
to
bc3331a
Compare
There was a problem hiding this 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)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (Code review)
This PR has the |
0147b8f
to
6b88e48
Compare
@pedrobaeza Can you update your review? Thanks! |
Please fix these flake8 erros:
|
6b88e48
to
00ccb79
Compare
…able to model table
00ccb79
to
cf92e08
Compare
@pedrobaeza sorry, done |
No description provided.