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] account_check_printing_report_base #725

Open
wants to merge 50 commits into
base: 16.0
Choose a base branch
from

Conversation

hildickethan
Copy link
Member

@pedrobaeza
Copy link
Member

/ocabot migration account_check_printing_report_base

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Apr 29, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Apr 29, 2024
14 tasks
@hildickethan hildickethan force-pushed the 16.0-mig-account_check_printing_report_base branch 2 times, most recently from 801bb1b to 40b63a6 Compare May 2, 2024 07:06
Copy link

@rmorant rmorant left a comment

Choose a reason for hiding this comment

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

Tested in my installation. It works. Thank you.

@AaronHForgeFlow
Copy link
Contributor

Hi @hildickethan , could you rebase the PR? runboat is not available

JordiBForgeFlow and others added 22 commits July 23, 2024 11:45
- account_check_printing_report_base
- account_check_printing_report_dlt103
…out verification from config settings, now the module use layout of journal if this have one defined, else use the company's one.
pedrocs-exo and others added 24 commits July 23, 2024 11:45
Currently translated at 92.9% (26 of 28 strings)

Translation: account-payment-12.0/account-payment-12.0-account_check_printing_report_base
Translate-URL: https://translation.odoo-community.org/projects/account-payment-12-0/account-payment-12-0-account_check_printing_report_base/pt/
Added the improvement that set the fix of spanish lang, and code to avoid reapeat it
on specific banks modules.
…ix typo in function call

The current version used and installed by Odoo is 0.5.6: https://github.com/odoo/odoo/blob/13.0/requirements.txt#L21
It has a typo in the function call to to_currency where "separator" is spelled "seperator".
As that is the version that Odoo uses, we should adapt to it, and also pin to ensure we are installing the correct one.

The typo is fixed in savoirfairelinux/num2words@2eee037 for version 0.5.10, so we should revert this if we ever upgrade to that version

TT30932
Changed the way to choose the layout of the check printing, by using the
way that odoo uses on core version, and extended the functionality
to allow choose that layout at journal level.

TT36415
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: account-payment-15.0/account-payment-15.0-account_check_printing_report_base
Translate-URL: https://translation.odoo-community.org/projects/account-payment-15-0/account-payment-15-0-account_check_printing_report_base/
Currently translated at 100.0% (26 of 26 strings)

Translation: account-payment-15.0/account-payment-15.0-account_check_printing_report_base
Translate-URL: https://translation.odoo-community.org/projects/account-payment-15-0/account-payment-15-0-account_check_printing_report_base/es/
@ioans73 ioans73 force-pushed the 16.0-mig-account_check_printing_report_base branch from 40b63a6 to 0c11e25 Compare July 23, 2024 09:45
@ioans73
Copy link
Member

ioans73 commented Jul 23, 2024

Hi @hildickethan , could you rebase the PR? runboat is not available

Done!

Copy link
Contributor

@AaronHForgeFlow AaronHForgeFlow left a comment

Choose a reason for hiding this comment

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

Functional tests

Copy link
Contributor

@AaronHForgeFlow AaronHForgeFlow left a comment

Choose a reason for hiding this comment

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

Can you please remove the migration scripts from older versions?

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.