-
-
Notifications
You must be signed in to change notification settings - Fork 314
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] edi_voxel_oca: Migration to version 15.0 #676
Conversation
[IMP] edi_voxel: Don't re-enqueue a new job if a failing one exists [FIX] edi_voxel: Remove failed jobs on cancel
[IMP] edi_voxel: Add voxel_enabled field in partners [IMP] edi_voxel*: use one login per service [FIX] edi_voxel: use document's company [IMP] edi_voxel: add methods to import documents from Voxel Usefull for edi_voxel_sale_order_import module.
[FIX] edi_voxel: ACL [FIX] edi_voxel: Security files always first [FIX+IMP] edi_voxel: Fine-tuning on the module [FIX] edi_voxel: Propagate company [FIX+IMP] edi_voxel: Raise if product not found + company UI reorder + play onchanges
[IMP] edi_voxel: Several things: * Move customer product SKU to main module for being used for all modules * Improve customerinfo search code [FIX] edi_voxel: Remove ETA It should be False, not 0 [IMP] edi_voxel: Remove "ES" from VAT if exists
…r (squashed) [FIX] edi_voxel: Trailing comma [FIX] pre-commit
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: edi-13.0/edi-13.0-edi_voxel Translate-URL: https://translation.odoo-community.org/projects/edi-13-0/edi-13-0-edi_voxel/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: edi-13.0/edi-13.0-edi_voxel Translate-URL: https://translation.odoo-community.org/projects/edi-13-0/edi-13-0-edi_voxel/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: edi-13.0/edi-13.0-edi_voxel Translate-URL: https://translation.odoo-community.org/projects/edi-13-0/edi-13-0-edi_voxel/
Why is it red? |
This module has not tests. The tests are in the other modules that extend the mixin |
/ocabot merge nobump |
This PR looks fantastic, let's merge it! |
@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 15.0-ocabot-merge-pr-676-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. |
@etobella @simahawk do you know about this error in https://github.com/OCA/edi/actions/runs/3496333620/jobs/5854123764#step:8:140 |
CI fixed with #683 |
/ocabot merge nobump |
On my way to merge this fine PR! |
Congratulations, your PR was merged at 6a20028. Thanks a lot for contributing to OCA. ❤️ |
cc @Tecnativa TT36454
ping @CarlosRoca13 @carlosdauden