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

17.0 mig l10n fi payment terms #71

Merged
merged 18 commits into from
Mar 10, 2025

Conversation

jarmokortetjarvi
Copy link

  • Migration from 14.0 to 17.0
  • Removed "Immediate payment", "21 Days", "30 Days", "45 Days" as redundant (included in core)
  • Removed the "Unique code" field that was added to payment terms. It may be useful, but this module isn't the best place to add the field.

Actual diff:
https://github.com/OCA/l10n-finland/compare/14.0...jarmokortetjarvi:l10n-finland:17.0-mig-l10n_fi_payment_terms?expand=1

@aisopuro
Copy link

You seem to have removed the .pot file, intentional?

Other than that seems fine. I believe @mlaitinen has merge access.

@jarmokortetjarvi jarmokortetjarvi force-pushed the 17.0-mig-l10n_fi_payment_terms branch from c6c6bf4 to 1c7227c Compare February 7, 2025 11:03
@jarmokortetjarvi
Copy link
Author

Recreated the missing .pot-file

@mlaitinen mlaitinen merged commit a449b36 into OCA:17.0 Mar 10, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants