account-invoicing icon indicating copy to clipboard operation
account-invoicing copied to clipboard

[MIG][15.0] account_invoicing_mode_at_shipping (renamed)

Open TDu opened this issue 3 years ago • 1 comments

This module migration depends on #1243

It also takes over this pr #1101

TDu avatar Sep 27 '22 13:09 TDu

@TDu To easy review, could you add the dependency PR link in this description ? Thanks

rousseldenis avatar Sep 29 '22 06:09 rousseldenis

/ocabot migration account_invoice_mode_at_shipping

rafaelbn avatar Dec 09 '22 19:12 rafaelbn

Can you remove the migration script here as well?

StefanRijnhart avatar Dec 12 '22 09:12 StefanRijnhart

@StefanRijnhart Done

TDu avatar Dec 12 '22 10:12 TDu

@pedrobaeza this one can either be merge :) thanks !

houssine78 avatar Apr 18 '23 15:04 houssine78

It can't be merged without removing the unreleased dependency

pedrobaeza avatar Apr 18 '23 15:04 pedrobaeza

@pedrobaeza the dependency is on the account_invoice_base_invoicing_mode which has been merged yesterday.

houssine78 avatar Apr 19 '23 08:04 houssine78

@houssine78 test-requirements.txt needs to be removed by the author @TDu.

StefanRijnhart avatar Apr 19 '23 09:04 StefanRijnhart

@StefanRijnhart I removed the test-requirements.txt change, should be good to merge.

TDu avatar Apr 20 '23 08:04 TDu

Thanks! /ocabot merge nobump

StefanRijnhart avatar Apr 20 '23 11:04 StefanRijnhart

This PR looks fantastic, let's merge it! Prepared branch 15.0-ocabot-merge-pr-1247-by-StefanRijnhart-bump-nobump, awaiting test results.

OCA-git-bot avatar Apr 20 '23 11:04 OCA-git-bot

Congratulations, your PR was merged at 563cd20f4e9b5501f86aa340ea2db2a7cceeda49. Thanks a lot for contributing to OCA. ❤️

OCA-git-bot avatar Apr 20 '23 11:04 OCA-git-bot