[16.0][ADD] purchase_planned_date_container_deposit: When date_planned is set manually, prevent calculation of the field up#2979
Conversation
|
@cyrilmanuel Do you want to merge? |
|
hi @pedrobaeza this pr can be merge by you ? |
|
This PR has the |
|
Such long names may have problems in certain publication pipelines. Can you short it? |
|
@chaule97 is it good for you to short that one ? |
78ec8c1 to
fd6ccd9
Compare
|
Hello @cyrilmanuel , @pedrobaeza , is this name okay? |
|
/ocabot merge nobump |
|
On my way to merge this fine PR! |
|
@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-2979-by-pedrobaeza-bump-nobump. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
fd6ccd9 to
34f1096
Compare
|
Hello @cyrilmanuel , about chrome test bug, what should I do ? |
|
hi @chaule97 same error as you on another pr, OCA/sale-workflow#4195 seems to be fixed now if we rebase the code :) |
34f1096 to
84f09f9
Compare
|
Hola @pedrobaeza , could you please merge again? |
|
/ocabot merge nobump |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
@cyrilmanuel wow we have another problem :) |
|
mmmm not sure if it's something wrong on the merge or if the manifest contain an error. @pedrobaeza do you have an idea ? |
|
I don't have idea I'm afraid. Have you do a pre-commit run to make sure everything related to setup is OK? |
84f09f9 to
27471e0
Compare
|
Hi @pedrobaeza , Can we try again? |
|
/ocabot merge nobump |
|
On my way to merge this fine PR! |
|
@pedrobaeza The merge process could not be finalized, because command |
|
/ocabot merge nobump |
|
This PR looks fantastic, let's merge it! |
|
@HviorForgeFlow The merge process could not be finalized, because command |
|
hi @chaule97 it seem's that the setup folder are wrong. perhaps because it dosen't contain the module. do you know how to generate that one ? ( i didn't create a module and create the setup part. i cannot help you on that) that seems to be wrongly generated on setup (normally we found a copy of the module) |
…et manually, prevent calculation of the field upon PO confirmation
27471e0 to
e75b958
Compare
|
Hi @pedrobaeza , Can we try again? |
|
pre-commit should auto-fix it. /ocabot merge nobump |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at 44665d7. Thanks a lot for contributing to OCA. ❤️ |
reopen: #2342