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

[18.0][MIG] l10n_ch_account_tags: Migration to 18.0 #743

Merged
merged 17 commits into from
Jan 28, 2025

Conversation

Kimkhoi3010
Copy link

@Kimkhoi3010 Kimkhoi3010 commented Nov 27, 2024

Since account.template is no longer supported from version 17.0, the process of linking account.tag to account.account will be handled using post_init.

mpanarin and others added 15 commits November 20, 2024 10:30
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-switzerland-12.0/l10n-switzerland-12.0-l10n_ch_account_tags
Translate-URL: https://translation.odoo-community.org/projects/l10n-switzerland-12-0/l10n-switzerland-12-0-l10n_ch_account_tags/
Split the csv by type of operation: update and new

In update reduce the number of columns to only set the tags
Column name and column user_type_id were overwriting
account template with bad values.

Also sort account.account.template in the csv file.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-switzerland-13.0/l10n-switzerland-13.0-l10n_ch_account_tags
Translate-URL: https://translation.odoo-community.org/projects/l10n-switzerland-13-0/l10n-switzerland-13-0-l10n_ch_account_tags/
@Kimkhoi3010 Kimkhoi3010 force-pushed the 18.0-mig-l10n_ch_account_tags branch from 8dcccb8 to 8df985b Compare November 27, 2024 02:51
@Kimkhoi3010 Kimkhoi3010 force-pushed the 18.0-mig-l10n_ch_account_tags branch from 8df985b to 7579c22 Compare November 27, 2024 02:54
Copy link
Member

@TDu TDu left a comment

Choose a reason for hiding this comment

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

Thanks for the migration, that migration script makes sense without the templates.

Copy link

@ecino ecino left a comment

Choose a reason for hiding this comment

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

LGTM

@ecino
Copy link

ecino commented Jan 28, 2025

/ocabot migration l10n_ch_account_tags

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Jan 28, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Jan 28, 2025
2 tasks
@ecino
Copy link

ecino commented Jan 28, 2025

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-743-by-ecino-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 5046695 into OCA:18.0 Jan 28, 2025
6 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 6bece36. 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.