[18.0][OU-ADD] project#5203
Conversation
|
/ocabot migration project |
| rule = env.ref( | ||
| "project_todo.task_visibility_rule_project_user", raise_if_not_found=False | ||
| ) | ||
| if rule: |
There was a problem hiding this comment.
@MiquelRForgeFlow you don't need to do this check, as simply calling the rename method will perform the same: if it doesn't exist, nothing happens.
There was a problem hiding this comment.
It's to avoid this log: 
I mean, if somebody doesn't have project_todo and see this log, they may wrongly worry why it's missing an xmlid.
There was a problem hiding this comment.
It's a info one, so no problem, and it's done this way across whole OpenUpgrade.
| new model project.share.collaborator.wizard [transient] | ||
| ---Fields in module 'project'--- | ||
| project / project.collaborator / limited_access (boolean) : NEW hasdefault: default | ||
| # NOTHING TO DO |
There was a problem hiding this comment.
Please put why is nothing to do.
There was a problem hiding this comment.
boolean with default False
There was a problem hiding this comment.
But please put it on the work file for other times, as stated several times. We don't know from this file which is the default and why it's correct to let it that way.
3 participants
Supersedes #5036.