[MIG] account_financial_risk: Migration to 14.0 #105

psugne · 2020-10-29T16:03:10Z

* [9.0][IMP] partner_financial_risk: Improve performance * [9.0][IMP] partner_financial_risk: Improve tests and partner hierarchy * [9.0][IMP] partner_stock_risk: Test

…ft_invoices

- Refunds adds partner risk instead of reduce. - If date_maturity changes to previous today date "never" compute partner risk.

* [9.0][IMP] partner_financial_risk: Improve multicompany cron * [9.0][IMP] partner_financial_risk: Exec with sudo (cherry picked from commit 6590bbf)

…idation

…ation wizard

Currently translated at 71.2% (52 of 73 strings) Translation: credit-control-12.0/credit-control-12.0-account_financial_risk Translate-URL: https://translation.odoo-community.org/projects/credit-control-12-0/credit-control-12-0-account_financial_risk/fr/

Currently translated at 79.5% (58 of 73 strings) Translation: credit-control-12.0/credit-control-12.0-account_financial_risk Translate-URL: https://translation.odoo-community.org/projects/credit-control-12-0/credit-control-12-0-account_financial_risk/it/

…o False.

Currently translated at 100.0% (73 of 73 strings) Translation: credit-control-12.0/credit-control-12.0-account_financial_risk Translate-URL: https://translation.odoo-community.org/projects/credit-control-12-0/credit-control-12-0-account_financial_risk/es/

Currently translated at 13.7% (10 of 73 strings) Translation: credit-control-12.0/credit-control-12.0-account_financial_risk Translate-URL: https://translation.odoo-community.org/projects/credit-control-12-0/credit-control-12-0-account_financial_risk/es_CL/

Currently translated at 76.7% (56 of 73 strings) Translation: credit-control-12.0/credit-control-12.0-account_financial_risk Translate-URL: https://translation.odoo-community.org/projects/credit-control-12-0/credit-control-12-0-account_financial_risk/pt_BR/

Currently translated at 100.0% (73 of 73 strings) Translation: credit-control-12.0/credit-control-12.0-account_financial_risk Translate-URL: https://translation.odoo-community.org/projects/credit-control-12-0/credit-control-12-0-account_financial_risk/pt/

…ption. Improve views and translations.

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: credit-control-12.0/credit-control-12.0-account_financial_risk Translate-URL: https://translation.odoo-community.org/projects/credit-control-12-0/credit-control-12-0-account_financial_risk/

…nfig settings

pedrobaeza · 2020-11-09T08:26:08Z

I'm seeing anyway that you haven't done the procedure depicted in https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-14.0 properly, as you have to pass pre-commit just after the commit history preservation. You should squash as well all your migration commits into one. Please do it.

account_financial_risk/security/ir.model.access.csv

alan196

Tested functionally and only one change requested

carlosdauden

Attend @alan196 request changes

pedrobaeza · 2020-11-20T10:39:08Z

Can you please squash migration commits?

alan196

C̶o̶u̶l̶d̶ ̶y̶o̶u̶ ̶p̶l̶e̶a̶s̶e̶ ̶c̶o̶n̶s̶i̶d̶e̶r̶ ̶t̶h̶i̶s̶ ̶P̶R̶ ̶h̶t̶t̶p̶s̶:̶/̶/̶g̶i̶t̶h̶u̶b̶.̶c̶o̶m̶/̶O̶C̶A̶/̶c̶r̶e̶d̶i̶t̶-̶c̶o̶n̶t̶r̶o̶l̶/̶p̶u̶l̶l̶/̶1̶1̶1̶ ̶i̶t̶ ̶j̶u̶s̶t̶ ̶g̶e̶t̶ ̶m̶e̶r̶g̶e̶d̶ ̶y̶e̶s̶t̶e̶r̶d̶a̶y̶ ̶a̶n̶d̶ ̶i̶t̶ ̶a̶f̶f̶e̶c̶t̶s̶ ̶a̶l̶s̶o̶ ̶V̶1̶4̶

pedrobaeza · 2020-11-20T14:21:52Z

But that PR is for another module, isn't it?

alan196

Sorry about that! I get confused 👍

OCA-git-bot · 2020-11-20T14:44:16Z

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

pedrobaeza · 2020-11-20T15:05:33Z

/ocabot merge nobump

OCA-git-bot · 2020-11-20T15:06:10Z

On my way to merge this fine PR!
Prepared branch 14.0-ocabot-merge-pr-105-by-pedrobaeza-bump-nobump, awaiting test results.

Signed-off-by pedrobaeza

pedrobaeza · 2020-11-20T18:42:31Z

/ocabot merge nobump

OCA-git-bot · 2020-11-20T18:42:47Z

What a great day to merge this nice PR. Let's do it!
Prepared branch 14.0-ocabot-merge-pr-105-by-pedrobaeza-bump-nobump, awaiting test results.

Signed-off-by pedrobaeza

pedrobaeza · 2020-11-26T16:58:29Z

/ocabot merge nobump

OCA-git-bot · 2020-11-26T16:59:04Z

What a great day to merge this nice PR. Let's do it!
Prepared branch 14.0-ocabot-merge-pr-105-by-pedrobaeza-bump-nobump, awaiting test results.

OCA-git-bot · 2020-11-26T17:15:32Z

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

carlosdauden and others added 30 commits October 28, 2020 09:39

[ADD] partner_financial_risk (OCA#292)

9b6245c

OCA Transbot updated translations from Transifex

2a72b54

[9.0][FIX] partner_financial_risk: Refund were not contemplated

649331a

OCA Transbot updated translations from Transifex

9a3d8ea

[9.0][IMP] partner_financial_risk: Improve performance (OCA#385)

ea75dcc

* [9.0][IMP] partner_financial_risk: Improve performance * [9.0][IMP] partner_financial_risk: Improve tests and partner hierarchy * [9.0][IMP] partner_stock_risk: Test

OCA Transbot updated translations from Transifex

7cc13fc

[9.0][FIX] partner_financial_risk: Error when risk_total includes dra…

c5e0f09

…ft_invoices

[9.0][IMP] partner_financial_risk: Add track visibility to credit limit

fa21079

[MIG] Partner Financial Risk : Migration to 10.0

9ce9449

[IMP] Minor changes

0af54c7

[FIX] partner_financial_risk: Invoice refunds add risk

74ca9bb

- Refunds adds partner risk instead of reduce. - If date_maturity changes to previous today date "never" compute partner risk.

[IMP] partner_financial_risk: New fields in view. Improve code.

97a356c

[10.0][IMP] partner_financial_risk: Improve multicompany cron (#530)

e431d06

* [9.0][IMP] partner_financial_risk: Improve multicompany cron * [9.0][IMP] partner_financial_risk: Exec with sudo (cherry picked from commit 6590bbf)

[MIG] partner_financial_risk: Migration to 11.0

72a8fa8

[REN] partner_financial_risk > account_financial_risk

b224e9e

[IMP] account_financial_risk: Improve table layout

e837a99

[IMP] account_financial_risk: Only block normal customer invoices val…

4f12767

…idation

[MIG] account_financial_risk: Migration to 12.0

bf5bb68

[IMP] account_financial_risk: Manage risk in multiple invoice confirm…

5dcd882

…ation wizard

[UPD] README.rst

6141ea7

[FIX]account_financial_risk: risk_exception field explicitly setted t…

855d5b9

…o False.

[IMP] account_financial_risk: Add allow overrisk invoice validation o…

823a04f

…ption. Improve views and translations.

[IMP] account_financial_risk: Avoid compute all partners when save co…

e68c40a

…nfig settings

pedrobaeza requested a review from carlosdauden November 9, 2020 08:25

psugne force-pushed the 14.0-mig-account_financial_risk branch from 175cbe9 to 5de11b0 Compare November 10, 2020 08:58

alan196 reviewed Nov 19, 2020

View reviewed changes

account_financial_risk/security/ir.model.access.csv Outdated Show resolved Hide resolved

alan196 suggested changes Nov 19, 2020

View reviewed changes

carlosdauden approved these changes Nov 20, 2020

View reviewed changes

[MIG] account_financial_risk: Migration to 14.0

5d95c65

psugne force-pushed the 14.0-mig-account_financial_risk branch from bc06233 to 5d95c65 Compare November 20, 2020 10:42

psugne requested a review from alan196 November 20, 2020 10:42

alan196 suggested changes Nov 20, 2020

View reviewed changes

alan196 approved these changes Nov 20, 2020

View reviewed changes

OCA-git-bot added approved ready to merge labels Nov 20, 2020

OCA-git-bot added a commit that referenced this pull request Nov 20, 2020

Merge PR #105 into 14.0

fed51b4

Signed-off-by pedrobaeza

OCA-git-bot added the bot is merging ⏳ label Nov 20, 2020

OCA-git-bot added a commit that referenced this pull request Nov 20, 2020

Merge PR #105 into 14.0

0c5ba2f

Signed-off-by pedrobaeza

pedrobaeza mentioned this pull request Nov 26, 2020

Migration to version 14.0 #96

Closed

6 tasks

OCA-git-bot merged commit cd832ce into OCA:14.0 Nov 26, 2020

OCA-git-bot added merged 🎉 and removed bot is merging ⏳ labels Nov 26, 2020

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[MIG] account_financial_risk: Migration to 14.0 #105

[MIG] account_financial_risk: Migration to 14.0 #105

psugne commented Oct 29, 2020

pedrobaeza commented Nov 9, 2020

alan196 left a comment

carlosdauden left a comment

pedrobaeza commented Nov 20, 2020

alan196 left a comment •

edited

Loading

pedrobaeza commented Nov 20, 2020

alan196 left a comment

OCA-git-bot commented Nov 20, 2020

pedrobaeza commented Nov 20, 2020

OCA-git-bot commented Nov 20, 2020

pedrobaeza commented Nov 20, 2020

OCA-git-bot commented Nov 20, 2020

pedrobaeza commented Nov 26, 2020

OCA-git-bot commented Nov 26, 2020

OCA-git-bot commented Nov 26, 2020

[MIG] account_financial_risk: Migration to 14.0 #105

[MIG] account_financial_risk: Migration to 14.0 #105

Conversation

psugne commented Oct 29, 2020

pedrobaeza commented Nov 9, 2020

alan196 left a comment

Choose a reason for hiding this comment

carlosdauden left a comment

Choose a reason for hiding this comment

pedrobaeza commented Nov 20, 2020

alan196 left a comment • edited Loading

Choose a reason for hiding this comment

pedrobaeza commented Nov 20, 2020

alan196 left a comment

Choose a reason for hiding this comment

OCA-git-bot commented Nov 20, 2020

pedrobaeza commented Nov 20, 2020

OCA-git-bot commented Nov 20, 2020

pedrobaeza commented Nov 20, 2020

OCA-git-bot commented Nov 20, 2020

pedrobaeza commented Nov 26, 2020

OCA-git-bot commented Nov 26, 2020

OCA-git-bot commented Nov 26, 2020

alan196 left a comment •

edited

Loading