-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[15.0][MIG] base_sparse_field_list_support #2371
[15.0][MIG] base_sparse_field_list_support #2371
Conversation
@HviorForgeFlow Small issue with pre-commit, should be easy to fix |
c8f7185
to
1a02c68
Compare
1a02c68
to
ab3e0a5
Compare
Hey @HviorForgeFlow, thank you for your Pull Request. It looks like some users haven't signed our Contributor License Agreement, yet.
Appreciation of efforts, |
ab3e0a5
to
d409ff7
Compare
d409ff7
to
b9cabd7
Compare
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
b9cabd7
to
e1f5c84
Compare
e1f5c84
to
f161540
Compare
This PR has the |
/ocabot merge nobump /ocabot migration base_sparse_field_support |
Hey, thanks for contributing! Proceeding to merge this for you. |
@thomaspaulb your merge command was aborted due to failed check(s), which you can inspect on this commit of 15.0-ocabot-merge-pr-2371-by-thomaspaulb-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. |
f161540
to
3763195
Compare
@thomaspaulb could you retry the merge please? |
/ocabot merge nobump |
This PR looks fantastic, let's merge it! |
@thomaspaulb your merge command was aborted due to failed check(s), which you can inspect on this commit of 15.0-ocabot-merge-pr-2371-by-thomaspaulb-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. |
* [ADD] restore travis configuration * [PEP] fix pep8 * [REF] exclude product_categ_available_pos of testing due to issue odoo/odoo#23749 * [PEP] fix pep8
…no more the object
3763195
to
22785b2
Compare
Rebased! |
@thomaspaulb can you please try again to merge? |
/ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at 48b7b09. Thanks a lot for contributing to OCA. ❤️ |
Standard migration
CC @ForgeFlow