Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[15.0][MIG] l10n_nl_tax_statement_icp #359

Merged
merged 19 commits into from
Nov 14, 2023
Merged

[15.0][MIG] l10n_nl_tax_statement_icp #359

merged 19 commits into from
Nov 14, 2023

Conversation

astirpe
Copy link
Member

@astirpe astirpe commented Aug 12, 2022

Depends on #355

@astirpe astirpe force-pushed the 15_mig_l10n_nl_tax_statement_icp branch 2 times, most recently from cfd149f to 50eab0a Compare August 12, 2022 10:31
@NL66278
Copy link
Contributor

NL66278 commented Oct 8, 2022

@astirpe Can you rebase this one and solve any remaining problems?

@astirpe astirpe force-pushed the 15_mig_l10n_nl_tax_statement_icp branch 2 times, most recently from 2ef16d3 to 09c0c4f Compare December 29, 2022 15:27
@astirpe astirpe marked this pull request as ready for review December 29, 2022 15:30
astirpe and others added 17 commits June 30, 2023 14:59
@astirpe
[11.0][ADD] l10n_nl_tax_statement_icp
389dfa9
@OCA-git-bot @astirpe
[UPD] README.rst
a805944
@oca-travis @astirpe
[UPD] Update l10n_nl_tax_statement_icp.pot
1b6e3d6
@OCA-git-bot @astirpe
[ADD] icon.png
2b41fc7
@OCA-git-bot @astirpe
[UPD] README.rst
9495bed
@astirpe
[12.0][MIG] l10n_nl_tax_statement_icp
eea5296
@astirpe
[13.0][MIG] l10n_nl_tax_statement_icp
a5fa270
@astirpe
Update readme
6c22b0a
@astirpe
Fix button name
bf788fb
@astirpe
Also calculate ICP lines in draft
155c4bb
@astirpe
[14.0][MIG] l10n_nl_tax_statement_icp
0ec977a
@oca-travis @astirpe
[UPD] Update l10n_nl_tax_statement_icp.pot
d2b82c5
@OCA-git-bot @astirpe
[UPD] README.rst
967ea6d
@bosd @astirpe
Added translation using Weblate (Dutch)
f0c9bf7
@bosd @astirpe
Translated using Weblate (Dutch)
b093479 
Currently translated at 100.0% (34 of 34 strings)

Translation: l10n-netherlands-14.0/l10n-netherlands-14.0-l10n_nl_tax_statement_icp
Translate-URL: https://translation.odoo-community.org/projects/l10n-netherlands-14-0/l10n-netherlands-14-0-l10n_nl_tax_statement_icp/nl/
@OCA-git-bot @astirpe
[IMP] update dotfiles [ci skip]
89cc8ac
@astirpe
[15.0][MIG] l10n_nl_tax_statement_icp
e2e5bae
@astirpe astirpe force-pushed the 15_mig_l10n_nl_tax_statement_icp branch from 09c0c4f to 4e451a0 Compare June 30, 2023 13:00
@astirpe astirpe force-pushed the 15_mig_l10n_nl_tax_statement_icp branch from 4e451a0 to 22a9e6b Compare July 5, 2023 11:57
@astirpe astirpe mentioned this pull request Jul 5, 2023
Merged
NL66278
NL66278 approved these changes Jul 6, 2023
View reviewed changes
Copy link
Contributor

@NL66278 NL66278 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 LGTM

@thomaspaulb
Copy link
Contributor

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 15.0-ocabot-merge-pr-359-by-thomaspaulb-bump-patch, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Oct 24, 2023
@OCA-git-bot
Merge PR #359 into 15.0
bcc7819 
Signed-off-by thomaspaulb
@OCA-git-bot
Copy link
Contributor

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). 🤖

@OCA-git-bot
Copy link
Contributor

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

@gjotten
Copy link

gjotten commented Nov 14, 2023

@thomaspaulb @NL66278 can either of you retry merging this?

@thomaspaulb
Copy link
Contributor

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit bea4ca4 into OCA:15.0 Nov 14, 2023
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thomaspaulb thomaspaulb thomaspaulb approved these changes

@NL66278 NL66278 NL66278 approved these changes

Assignees
No one assigned
Labels
approved merged 🎉 ready to merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@astirpe @NL66278 @thomaspaulb @OCA-git-bot @gjotten @oca-travis @bosd