-
-
Notifications
You must be signed in to change notification settings - Fork 699
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] hr #2789
[14.0][MIG] hr #2789
Conversation
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
d186870
to
bfcbf53
Compare
Hi @MiquelRForgeFlow , I'm not sure if it should be there but I faced the following error using your scripts :
In v14 the hr.res_users_view_form_simple_modif view has no more the same usage and res_users_view_form_profile is pointing to another view https://github.com/OCA/OCB/blob/14.0/addons/mail_bot_hr/views/res_users_views.xml#L17 What is your recommandation ? |
another one
|
That one we fixed it with ForgeFlow/odoo@c80f3cb, still waiting odoo to accept the corresponding PR |
The other one fixed with ForgeFlow/odoo@b8d215c. |
bfcbf53
to
e746914
Compare
- Add more comments to the analysis work file - Refine company_id assignation using department's company first - Don't load noupdate records without changes TT29982
e746914
to
5a695d5
Compare
Link to code does not work. I have the same error as @flotho has. |
execute this query before migration and you will avoid this stupid view errors DELETE FROM ir_ui_view iuv
USING ir_model_data imd
WHERE imd.model = 'ir.ui.view' AND imd.res_id = iuv.id AND imd.noupdate IS DISTINCT FROM TRUE |
Supersedes #2777 (I couldn't edit that PR).