[16.0][OU-ADD] website_sale_loyalty_delivery: Migration to version 16.0 #4527
Conversation
This file contains 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
chienandalu
approved these changes
Jul 17, 2024
|
This is not |
|
/ocabot migration website_sale_loyalty_delivery Depends on :
|
pilarvargas-tecnativa
force-pushed
the
16.0-mig-website_sale_loyalty_delivery
branch
from
dc0a573 to
0440e7e
Compare
pilarvargas-tecnativa
changed the title
|
@pedrobaeza done! thanks! |
pedrobaeza
reviewed
Jul 18, 2024
pilarvargas-tecnativa
force-pushed
the
16.0-mig-website_sale_loyalty_delivery
branch
from
0440e7e to
eb20029
Compare
pedrobaeza
reviewed
Jul 18, 2024
pilarvargas-tecnativa
force-pushed
the
16.0-mig-website_sale_loyalty_delivery
branch
from
eb20029 to
fe3b01b
Compare
pilarvargas-tecnativa
force-pushed
the
16.0-mig-website_sale_loyalty_delivery
branch
from
fe3b01b to
6447b86
Compare
|
This PR looks fantastic, let's merge it! |
OCA-git-bot
added a commit
that referenced
this pull request
Jul 18, 2024
|
@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-4527-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. |
pilarvargas-tecnativa
force-pushed
the
16.0-mig-website_sale_loyalty_delivery
branch
from
c5e9df3 to
3072f27
Compare
pilarvargas-tecnativa
force-pushed
the
16.0-mig-website_sale_loyalty_delivery
branch
4 times, most recently
from
ddaeb9e to
23aca54
Compare
pedrobaeza
reviewed
Jul 19, 2024
openupgrade_scripts/scripts/sale_loyalty/16.0.1.0/post-migration.py
Outdated
Show resolved
Hide resolved
pilarvargas-tecnativa
force-pushed
the
16.0-mig-website_sale_loyalty_delivery
branch
from
23aca54 to
b8f11da
Compare
This script is not necessary since both the table and the constraint are new in v16, it is only necessary to apply this restriction to the data dump in the new table by adding a condition that contemplates it.
pilarvargas-tecnativa
force-pushed
the
16.0-mig-website_sale_loyalty_delivery
branch
from
b8f11da to
ff2ee1c
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
cc @Tecnativa TT49821
@pedrobaeza @chienandalu please review