Merged
Conversation
Closed
dennybiasiolli
approved these changes
Aug 14, 2020
dennybiasiolli
suggested changes
Aug 14, 2020
dennybiasiolli
left a comment
dennybiasiolli
left a comment
There was a problem hiding this comment.
Update break if mrp_subproduct was not installed in 12.0.
Take a look to the suggested solution
kos94ok-3D
force-pushed
the
13.0-mig-mrp
branch
3 times, most recently
from
0246a6e to
7a52577
Compare
dennybiasiolli
approved these changes
Aug 17, 2020
dennybiasiolli
suggested changes
Nov 17, 2020
kos94ok-3D
force-pushed
the
13.0-mig-mrp
branch
2 times, most recently
from
aa14dea to
efeb465
Compare
dennybiasiolli
suggested changes
Nov 18, 2020
kos94ok-3D
force-pushed
the
13.0-mig-mrp
branch
2 times, most recently
from
ee41f6a to
ad77122
Compare
MiquelRForgeFlow
force-pushed
the
13.0-mig-mrp
branch
2 times, most recently
from
5cdd22a to
d1afb3d
Compare
MiquelRForgeFlow
force-pushed
the
13.0-mig-mrp
branch
from
d1afb3d to
9488af7
Compare
dennybiasiolli
added a commit
to SequarSrl/OpenUpgrade
that referenced
this pull request
Dec 7, 2020
Author
|
Hi, @MiquelRForgeFlow. |
Contributor
|
done |
MiquelRForgeFlow
force-pushed
the
13.0-mig-mrp
branch
from
9488af7 to
4f2aeef
Compare
Author
|
Rebase and diff |
MiquelRForgeFlow
force-pushed
the
13.0-mig-mrp
branch
from
54c9278 to
c67ea77
Compare
Contributor
|
@kos94ok-3D if you don't mind, I added a commit to move the filling of the m2m table to post-migration. This way the old table still exists after the migration (and I need that table to test some things). |
MiquelRForgeFlow
force-pushed
the
13.0-mig-mrp
branch
2 times, most recently
from
79e4e6d to
b99f591
Compare
This was referenced May 6, 2021
Merged
MiquelRForgeFlow
force-pushed
the
13.0-mig-mrp
branch
2 times, most recently
from
658ea4d to
3ed9ab7
Compare
MiquelRForgeFlow
force-pushed
the
13.0-mig-mrp
branch
5 times, most recently
from
02e8578 to
0e70d57
Compare
MiquelRForgeFlow
force-pushed
the
13.0-mig-mrp
branch
from
0e70d57 to
c338bd2
Compare
MiquelRForgeFlow
approved these changes
May 31, 2021
Contributor
MiquelRForgeFlow
left a comment
MiquelRForgeFlow
left a comment
There was a problem hiding this comment.
Tested last changes. To me it's good this PR to merge.
kos94ok-3D
commented
May 31, 2021
Member
|
Not checked at all, but reviewed by some of you and run in a DB without errors, so merging for having a working version. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Supersedes #2298.
--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr