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.

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] l10n_nl_tax_statement #372

Merged
merged 49 commits into from
Jul 12, 2023

Conversation

astirpe
Copy link
Member

@astirpe astirpe commented Jan 18, 2023

No description provided.

@NL66278
Copy link
Contributor

NL66278 commented Jan 23, 2023

/ocabot migration l10n_nl_tax_statement

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Jan 23, 2023
@OCA-git-bot OCA-git-bot mentioned this pull request Jan 23, 2023
6 tasks
@OCA OCA deleted a comment from OCA-git-bot Jan 23, 2023
@NL66278
Copy link
Contributor

NL66278 commented Feb 22, 2023

@astirpe Please squash the consecutive translation commits.

@astirpe astirpe force-pushed the 16_mig_l10n_nl_tax_statement branch 2 times, most recently from 26de8c8 to efeef9e Compare February 23, 2023 08:04
astirpe and others added 22 commits February 24, 2023 10:02
* Add module l10n_nl_tax_statement

* Update oca_dependencies and flake8

* Developing module

* Decoupled from l10n_nl + added config feature

* Fix missing tag (Voorbelasting (BTW) bis)

* Add printable PDF report

* Add onchange for filling statement name

* Update layout

* Add tests

* Set postgres V9.2 for running tests with date_range

* Add nl.po translation file

* Fix 5a sum

* Update domain according with updated account_tax_balance

* Fix calculation of lines + minor review

* Update steps regarding Usage in README.rst
* Missing Dutch Translations

* Fix typo
…ort (OCA#107)

* Add editable fields: '5d', '5e', '5f

* Flake8 + pylint

* Fix E501 line too long

* Update tests

* Update version in manifest
astirpe and others added 14 commits February 24, 2023 10:02
[IMP] merge nl_NL.po into nl.po
Currently translated at 100.0% (113 of 113 strings)

Translation: l10n-netherlands-14.0/l10n-netherlands-14.0-l10n_nl_tax_statement
Translate-URL: https://translation.odoo-community.org/projects/l10n-netherlands-14-0/l10n-netherlands-14-0-l10n_nl_tax_statement/nl/
[MIG] l10n_nl_tax_statement: Migration to 15.0

[MIG] l10n_nl_tax_statement: Migration to 15.0

[IMP] l10n_nl_tax_statement: allow for Billing Administrator group

[FIX] l10n_nl_tax_statement: layout PDF
@astirpe astirpe force-pushed the 16_mig_l10n_nl_tax_statement branch from efeef9e to 922477c Compare February 24, 2023 09:02
Copy link

@CasVissers-360ERP CasVissers-360ERP left a comment

Choose a reason for hiding this comment

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

Migration review

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

@astirpe
Copy link
Member Author

astirpe commented Jun 30, 2023

Included #394

@NL66278
Copy link
Contributor

NL66278 commented Jul 12, 2023

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 16.0-ocabot-merge-pr-372-by-NL66278-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 05a32bc into OCA:16.0 Jul 12, 2023
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at e1758ac. 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants