OCA / l10n-switzerland

Odoo Swiss localization
GNU Affero General Public License v3.0
53 stars 164 forks source link

[15.0][MIG] l10n_ch_invoice_reports: Migrate to version 15.0 #660

Closed sonhd91 closed 2 years ago

TDu commented 2 years ago

Looks good, thank you. Also there was this fix https://github.com/OCA/l10n-switzerland/pull/656 that landed in the 13.0 version. It would be good to include it. What do you think @ajaniszewska-dev ?

sonhd91 commented 2 years ago

Looks good, thank you. Also there was this fix #656 that landed in the 13.0 version. It would be good to include it. What do you think @ajaniszewska-dev ?

I just included PR#656 in this PR, please check again

OCA-git-bot commented 2 years ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

ajaniszewska-dev commented 2 years ago

@simahawk could you please merge that one? Tested on lab - works fine.

simahawk commented 2 years ago

/ocabot migration l10n_ch_invoice_reports

OCA-git-bot commented 2 years ago

There's no issue in this repo with the title 'Migration to version 15.0' and the milestone 15.0, so not possible to add the comment.

simahawk commented 2 years ago

/ocabot merge nobump

OCA-git-bot commented 2 years ago

What a great day to merge this nice PR. Let's do it! Prepared branch 15.0-ocabot-merge-pr-660-by-simahawk-bump-nobump, awaiting test results.

OCA-git-bot commented 2 years ago

@simahawk your merge command was aborted due to failed check(s), which you can inspect on this commit of 15.0-ocabot-merge-pr-660-by-simahawk-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.

OCA-git-bot commented 2 years ago

@simahawk your merge command was aborted due to failed check(s), which you can inspect on this commit of 15.0-ocabot-merge-pr-660-by-simahawk-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.

simahawk commented 2 years ago

must merge this and rebase https://github.com/OCA/l10n-switzerland/pull/681

simahawk commented 2 years ago

/ocabot rebase

simahawk commented 2 years ago

@sonhd91 can you rebase pls?

sonhd91 commented 2 years ago

@sonhd91 can you rebase pls?

I just rebase it, thank you

simahawk commented 2 years ago

/ocabot merge nobump

OCA-git-bot commented 2 years ago

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

OCA-git-bot commented 2 years ago

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