-
Notifications
You must be signed in to change notification settings - Fork 15
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
[16.0][MIG] recurring_consignment_pos: Migration to 16.0 (from 12.0) #145
[16.0][MIG] recurring_consignment_pos: Migration to 16.0 (from 12.0) #145
Conversation
/ocabot migration recurring_consignment_pos |
5ae1bef
to
0839aaf
Compare
2368fc7
to
9e96309
Compare
/ocabot rebase |
Congratulations, PR rebased to 16.0. |
9e96309
to
dcda8c7
Compare
MeP du 02/07/2024 /ocabot merge patch |
Hey, thanks for contributing! Proceeding to merge this for you. |
@legalsylvain your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-145-by-legalsylvain-bump-patch. 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. |
/ocabot rebase |
Congratulations, PR rebased to 16.0. |
/ocabot rebase |
Congratulations, PR rebased to 16.0. |
dcda8c7
to
04e3d7f
Compare
/ocabot rebase |
Congratulations, PR rebased to 16.0. |
04e3d7f
to
0784450
Compare
0784450
to
1003e1f
Compare
… avoid to multiply by 2, the sold products, in the report
…r lines related to a commission invoice
- no need to use anymore constrains to prevent sale to customer in Point of sale - Add test - add constrains to avoid to change a consignor setting, if a PoS session is opened because incorrect taxes configuration should be loaded and then generate bad account move. [FIX] recurring_consignment: - Fix bad lines in commission report, for account move lines that are generated by PoS
d0cb137
to
5f1ae0c
Compare
…f reloading recurring_consigment demo data. Use correct payment method on recurring_consignment_pos module, removing hardcoded value.
5f1ae0c
to
bdb5b0b
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## 16.0 #145 +/- ##
==========================================
+ Coverage 90.06% 90.52% +0.45%
==========================================
Files 119 127 +8
Lines 1409 1530 +121
Branches 129 134 +5
==========================================
+ Hits 1269 1385 +116
- Misses 103 108 +5
Partials 37 37 ☔ View full report in Codecov by Sentry. |
PR déployé il y a plusieurs mois, mais les tests merdaient. /ocabot merge minor |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at 076c541. Thanks a lot for contributing to grap. ❤️ |
No description provided.