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

[FW][FIX] account_interests: Fix when having interval to run interest #189

Closed

Conversation

fw-bot-adhoc
Copy link
Contributor

Se realiza el arreglo de cuando se tiene un intervalo sobre el cual calcular.
Ese intervalo afecta a la deuda de periodos anteriores.

Por ejemplo si se tiene un interes mensual de 0,0068 diario y quiere que se repita cada 5 dias. Entonces a la hora de calcular el interes del perior anterio deberia ser 0,0068 int/dias * 5 dias * deuda

por otro lado, se quita el -1 ya que si se esta incluyenda la fecha de vencimiento entonces lo que ocurria es los siguiente
factura que vence el 15/10 y la corrida es del 16/10
entraba dado ('date_maturity', '>=', from_date) y esta resta (to_date - move.invoice_date_due).days) claramente daba 1 y al restarle luego 1 quedaba en 0 y nunca se cobra ese interes por ese dia de vencimiento

Forward-Port-Of: #186

@roboadhoc
Copy link

@fw-bot-adhoc
Copy link
Contributor Author

@rov-adhoc @ica-adhoc this PR targets 18.0 and is the last of the forward-port chain.

To merge the full chain, use

@fw-bot-adhoc r+

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@fw-bot-adhoc
Copy link
Contributor Author

@rov-adhoc @ica-adhoc ci/runbot-smarted failed on this forward-port PR

@rov-adhoc rov-adhoc force-pushed the 18.0-17.0-T-47626-rov-YLPI-fw branch from 8a673d8 to 38dae95 Compare January 22, 2025 11:25
@fw-bot-adhoc
Copy link
Contributor Author

@rov-adhoc @ica-adhoc this PR was modified / updated and has become a normal PR. It should be merged the normal way (via @roboadhoc)

@rov-adhoc
Copy link
Contributor

@roboadhoc r+

1 similar comment
@rov-adhoc
Copy link
Contributor

@roboadhoc r+

@roboadhoc
Copy link

@rov-adhoc you may want to rebuild or fix this PR as it has failed CI.

@roboadhoc
Copy link

I'm sorry, @rov-adhoc: this PR is already reviewed, reviewing it again is useless.

@rov-adhoc
Copy link
Contributor

@roboadhoc r+

@roboadhoc
Copy link

@rov-adhoc you may want to rebuild or fix this PR as it has failed CI.

@roboadhoc
Copy link

I'm sorry, @rov-adhoc: this PR is already reviewed, reviewing it again is useless.

@rov-adhoc rov-adhoc force-pushed the 18.0-17.0-T-47626-rov-YLPI-fw branch 2 times, most recently from a29b184 to e4da670 Compare January 22, 2025 14:44
@rov-adhoc
Copy link
Contributor

@roboadhoc r+

1 similar comment
@rov-adhoc
Copy link
Contributor

@roboadhoc r+

@roboadhoc
Copy link

@rov-adhoc you may want to rebuild or fix this PR as it has failed CI.

@roboadhoc
Copy link

I'm sorry, @rov-adhoc: this PR is already reviewed, reviewing it again is useless.

@rov-adhoc rov-adhoc force-pushed the 18.0-17.0-T-47626-rov-YLPI-fw branch from a11540f to f11b2d5 Compare January 24, 2025 17:00
@rov-adhoc
Copy link
Contributor

@roboadhoc r+

@roboadhoc
Copy link

@rov-adhoc @ica-adhoc linked pull request(s) ingadhoc/account-financial-tools#654 not ready. Linked PRs are not staged until all of them are ready.

@rov-adhoc
Copy link
Contributor

@roboadhoc r+

@roboadhoc
Copy link

I'm sorry, @rov-adhoc: this PR is already reviewed, reviewing it again is useless.

@rov-adhoc
Copy link
Contributor

@roboadhoc r+

1 similar comment
@rov-adhoc
Copy link
Contributor

@roboadhoc r+

@roboadhoc
Copy link

I'm sorry, @rov-adhoc: this PR is already reviewed, reviewing it again is useless.

1 similar comment
@roboadhoc
Copy link

I'm sorry, @rov-adhoc: this PR is already reviewed, reviewing it again is useless.

@rov-adhoc rov-adhoc force-pushed the 18.0-17.0-T-47626-rov-YLPI-fw branch from f11b2d5 to 9e159c0 Compare January 27, 2025 12:29
@rov-adhoc
Copy link
Contributor

@roboadhoc r+

@rov-adhoc rov-adhoc force-pushed the 18.0-17.0-T-47626-rov-YLPI-fw branch from 9e159c0 to d3cd70a Compare January 30, 2025 10:51
@rov-adhoc
Copy link
Contributor

@roboadhoc r+

roboadhoc pushed a commit to ingadhoc/account-financial-tools that referenced this pull request Jan 30, 2025
closes #654

X-original-commit: d7817f6
Related: ingadhoc/odoo-academic#189
Signed-off-by: Ignacio Cainelli <[email protected]>
Signed-off-by: rov-adhoc <[email protected]>
@roboadhoc roboadhoc closed this in 06794f0 Jan 30, 2025
@roboadhoc roboadhoc added the 17.1 label Jan 30, 2025
rov-adhoc added a commit to adhoc-dev/account-financial-tools that referenced this pull request Jan 30, 2025
closes ingadhoc#654

X-original-commit: d7817f6
Related: ingadhoc/odoo-academic#189
Signed-off-by: Ignacio Cainelli <[email protected]>
Signed-off-by: rov-adhoc <[email protected]>
rov-adhoc added a commit to adhoc-dev/account-financial-tools that referenced this pull request Jan 30, 2025
closes ingadhoc#654

X-original-commit: d7817f6
Related: ingadhoc/odoo-academic#189
Signed-off-by: Ignacio Cainelli <[email protected]>
Signed-off-by: rov-adhoc <[email protected]>
rov-adhoc added a commit to adhoc-dev/odoo-academic that referenced this pull request Jan 30, 2025
rov-adhoc added a commit to adhoc-dev/odoo-academic that referenced this pull request Jan 30, 2025
rov-adhoc added a commit to adhoc-dev/odoo-academic that referenced this pull request Jan 30, 2025
rov-adhoc added a commit to adhoc-dev/account-financial-tools that referenced this pull request Jan 30, 2025
closes ingadhoc#654

X-original-commit: d7817f6
Related: ingadhoc/odoo-academic#189
Signed-off-by: Ignacio Cainelli <[email protected]>
Signed-off-by: rov-adhoc <[email protected]>
roboadhoc pushed a commit that referenced this pull request Jan 30, 2025
closes #189

closes #193

X-original-commit: 04cdd73
Related: ingadhoc/account-financial-tools#654
Related: ingadhoc/account-financial-tools#659
Signed-off-by: Ignacio Cainelli <[email protected]>
Signed-off-by: Camila Vives <[email protected]>
Signed-off-by: rov-adhoc <[email protected]>
roboadhoc pushed a commit to ingadhoc/account-financial-tools that referenced this pull request Jan 30, 2025
closes #654

closes #659

X-original-commit: d7817f6
Related: ingadhoc/odoo-academic#189
Related: ingadhoc/odoo-academic#193
Signed-off-by: Ignacio Cainelli <[email protected]>
Signed-off-by: Camila Vives <[email protected]>
Signed-off-by: rov-adhoc <[email protected]>
rov-adhoc added a commit to adhoc-dev/account-financial-tools that referenced this pull request Jan 30, 2025
closes ingadhoc#654

closes ingadhoc#659

X-original-commit: d7817f6
Related: ingadhoc/odoo-academic#189
Related: ingadhoc/odoo-academic#193
Signed-off-by: Ignacio Cainelli <[email protected]>
Signed-off-by: Camila Vives <[email protected]>
Signed-off-by: rov-adhoc <[email protected]>
rov-adhoc added a commit to adhoc-dev/account-financial-tools that referenced this pull request Jan 30, 2025
closes ingadhoc#654

closes ingadhoc#659

X-original-commit: d7817f6
Related: ingadhoc/odoo-academic#189
Related: ingadhoc/odoo-academic#193
Signed-off-by: Ignacio Cainelli <[email protected]>
Signed-off-by: Camila Vives <[email protected]>
Signed-off-by: rov-adhoc <[email protected]>
rov-adhoc added a commit to adhoc-dev/account-financial-tools that referenced this pull request Jan 30, 2025
closes ingadhoc#654

closes ingadhoc#659

X-original-commit: d7817f6
Related: ingadhoc/odoo-academic#189
Related: ingadhoc/odoo-academic#193
Signed-off-by: Ignacio Cainelli <[email protected]>
Signed-off-by: Camila Vives <[email protected]>
Signed-off-by: rov-adhoc <[email protected]>
roboadhoc pushed a commit to ingadhoc/account-financial-tools that referenced this pull request Jan 30, 2025
closes #654

closes #659

closes #663

X-original-commit: d7817f6
Related: ingadhoc/odoo-academic#189
Related: ingadhoc/odoo-academic#193
Signed-off-by: Ignacio Cainelli <[email protected]>
Signed-off-by: Camila Vives <[email protected]>
Signed-off-by: rov-adhoc <[email protected]>
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.

3 participants