-
Notifications
You must be signed in to change notification settings - Fork 23
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
[MIG][17.0] account: migration to 17.0 #652
[MIG][17.0] account: migration to 17.0 #652
Conversation
e88c14f
to
ad3183a
Compare
ad3183a
to
f651f5a
Compare
todo:
|
f651f5a
to
8026a15
Compare
7f33c61
to
d9da64e
Compare
1120961
to
8f38706
Compare
openupgrade_scripts/scripts/account/17.0.1.2/upgrade_analysis_work.txt
Outdated
Show resolved
Hide resolved
openupgrade_scripts/scripts/account/17.0.1.2/upgrade_analysis_work.txt
Outdated
Show resolved
Hide resolved
openupgrade_scripts/scripts/account/17.0.1.2/upgrade_analysis_work.txt
Outdated
Show resolved
Hide resolved
c2fcbae
to
f1a4091
Compare
f1a4091
to
6800ee3
Compare
6800ee3
to
4f4015a
Compare
5fb854f
to
ebad039
Compare
SET tax_group_id = {new_tax_group_id} | ||
WHERE tax_group_id = {tax_group_id} AND company_id = {company_id} | ||
""", | ||
) |
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.
@duong77476-viindoo anh có sửa lại code để nó đơn giản hơn, vẫn theo logic của em, nó sẽ như này
đồng thời khi copy tax group thì anh không fix trường, vì nếu module nào đó thêm trường thì sẽ không thể kế thừa được
lấy tất cả tax_group_id từ account_tax để biết xem cái nào cần tạo mới cái nào cần nhân bản (cái đầu tiên sẽ luôn đổi xml, các cái tiếp theo sẽ nhân bản từ cái thứ nhất)
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
Depend: #654
https://viindoo.com/web#id=90693&cids=1&menu_id=289&action=409&active_id=475&model=project.task&view_type=form