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_bank #386

Merged
merged 21 commits into from
Mar 12, 2023
Merged

Conversation

astirpe
Copy link
Member

@astirpe astirpe commented Mar 7, 2023

No description provided.

ploegvde and others added 21 commits March 7, 2023 14:35
* New module l10n_nl_banks

* Remove blanc sections

* Icon credits

* OCA Requested changes

* Change module name

* Changes to comply with OCA
corrected country code

Remove account dependency

Fix migraton script

fix pre-commit

update res_bank_data.xml as of 23-07-2020

corrected country code

Remove account dependency

Fix migraton script

fix pre-commit
[REM] Old migrations

[IMP] l10n_nl_bank: black, isort, prettier

[IMP] l10n_nl_bank: black, isort, prettier
@astirpe astirpe force-pushed the 16_mig_l10n_nl_bank branch from 2643b27 to 51fce46 Compare March 7, 2023 13:42
@astirpe astirpe marked this pull request as ready for review March 7, 2023 13:42
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

Copy link
Contributor

@bosd bosd left a comment

Choose a reason for hiding this comment

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

LGTM 👍

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

@NL66278
Copy link
Contributor

NL66278 commented Mar 12, 2023

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-386-by-NL66278-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 46bc849 into OCA:16.0 Mar 12, 2023
@OCA-git-bot
Copy link
Contributor

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

@thomaspaulb
Copy link
Contributor

/ocabot migration l10n_nl_bank

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Nov 7, 2023
@OCA-git-bot OCA-git-bot mentioned this pull request Nov 7, 2023
6 tasks
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.

9 participants