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_payment_term_extension #580

Merged

Conversation

anothingguy
Copy link

  • since v16, Odoo now supports months natively, but not weeks and not some other features of this module, so it's still worth migrating it
  • option after_invoice_month can be used alternatively with flag end_month in v16

Alexis de Lattre and others added 30 commits January 18, 2023 11:18
…_term_extension

Add support for months and weeks in account.payment.term.line
Add unittest
PEP8 validation
Fix and add a feature for apply a chronological order on lines
Merge of the 'account_payment_term_multi_day' module
Add icon to module
[UPD] Update account_payment_term_extension.pot
Currently translated at 100.0% (28 of 28 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_payment_term_extension/es/
Currently translated at 100.0% (28 of 28 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_payment_term_extension/pt_BR/
Currently translated at 100.0% (28 of 28 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_payment_term_extension/fr/
Currently translated at 100.0% (28 of 28 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_payment_term_extension/fr_CH/
Currently translated at 85.7% (24 of 28 strings)

Translation: account-invoicing-12.0/account-invoicing-12.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-invoicing-12-0/account-invoicing-12-0-account_payment_term_extension/hr/
Currently translated at 100.0% (28 of 28 strings)

Translation: account-payment-13.0/account-payment-13.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-payment-13-0/account-payment-13-0-account_payment_term_extension/es_AR/
…re resilient

* A `account.journal` record was being searched and used to create invoices, even though it was not necessary.
 This introduced the problem of finding more than one record that satisfied the search query, breaking the test for a singleton error.
 This removes the need for that.
* Extract a method for creating the invoice.
* Don't depend on variable year for tests.

TT27288

Co-Authored-By: Pedro M. Baeza <[email protected]>
* Standard procedure
* Adapt code
* Adapt tests
* Convert tests to SavepointCase

[UPD] Update account_payment_term_extension.pot
OCA-git-bot and others added 4 commits January 18, 2023 11:27
Currently translated at 100.0% (34 of 34 strings)

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

Translation: account-payment-15.0/account-payment-15.0-account_payment_term_extension
Translate-URL: https://translation.odoo-community.org/projects/account-payment-15-0/account-payment-15-0-account_payment_term_extension/pt_BR/
@anothingguy anothingguy mentioned this pull request Feb 1, 2023
14 tasks
@pablontura
Copy link

Functional validation LGTM

@leemannd
Copy link

Hello @anothingguy , could you have a look at the pre-commit?

@cyrilmanuel
Copy link

LGTM :
pre-commit can be resolve regarding this :

OCA/oca-addons-repo-template#185

create new PR: checkout the branch (eg: 16.0) fork it and run copier -f update

@StephaneMangin
Copy link

Hello @anothingguy , could you have a look at the pre-commit please?

@anothingguy anothingguy force-pushed the 16.0-mig-account_payment_term_extension branch from 7dd0a1b to fe99ad1 Compare March 9, 2023 10:03
@anothingguy
Copy link
Author

Hello @anothingguy , could you have a look at the pre-commit please?

Hello @StephaneMangin, pre-commit is green now

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

@simahawk simahawk changed the title 16.0 mig account payment term extension [16.0][MIG] account_payment_term_extension Mar 9, 2023
@simahawk
Copy link

simahawk commented Mar 9, 2023

@anothingguy next time, please, fix the module name in the title of the PR: it makes easier to find it while searching for PRs.

/ocabot migration account_payment_term_extension

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Mar 9, 2023
@simahawk
Copy link

simahawk commented Mar 9, 2023

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-580-by-simahawk-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 460e8c3 into OCA:16.0 Mar 9, 2023
@OCA-git-bot
Copy link
Contributor

@simahawk The merge process could not be finalized, because command git push origin 16.0-ocabot-merge-pr-580-by-simahawk-bump-nobump:16.0 failed with output:

To https://github.com/OCA/account-payment
 ! [rejected]        16.0-ocabot-merge-pr-580-by-simahawk-bump-nobump -> 16.0 (fetch first)
error: failed to push some refs to 'https://***@github.com/OCA/account-payment'
hint: Updates were rejected because the remote contains work that you do
hint: not have locally. This is usually caused by another repository pushing
hint: to the same ref. You may want to first integrate the remote changes
hint: (e.g., 'git pull ...') before pushing again.
hint: See the 'Note about fast-forwards' in 'git push --help' for details.

@OCA-git-bot
Copy link
Contributor

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

@rafaelbn
Copy link
Member

rafaelbn commented Aug 4, 2023

This migration has an issue , we are loosing functionality:

#656

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.