Skip to content

[ADD] account_invoice_move_currency: exchange rate on customer invoices#251

Closed
cav-adhoc wants to merge 1 commit intoingadhoc:18.0from
adhoc-dev:18.0-t-63427-cav
Closed

[ADD] account_invoice_move_currency: exchange rate on customer invoices#251
cav-adhoc wants to merge 1 commit intoingadhoc:18.0from
adhoc-dev:18.0-t-63427-cav

Conversation

@cav-adhoc
Copy link
Copy Markdown
Contributor

This commit adds a new module 'account_invoice_move_currency' that allows users to specify a custom exchange rate on customer invoices. The module introduces two new fields on the account.move model: 'user_secondary_currency_id' for selecting a secondary currency, and 'user_exchange_rate' for entering the exchange rate. The exchange rate field is automatically computed based on the selected secondary currency and the invoice date, but remains editable for user input.

Copilot AI review requested due to automatic review settings February 3, 2026 17:46
@roboadhoc
Copy link
Copy Markdown
Contributor

Pull request status dashboard

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Este PR agrega un nuevo módulo account_invoice_move_currency que permite a los usuarios especificar un tipo de cambio personalizado con fines informativos en las facturas de cliente. El módulo introduce dos campos nuevos en account.move: user_secondary_currency_id para seleccionar una moneda secundaria de referencia, y user_exchange_rate para mostrar/editar el tipo de cambio entre esa moneda y la moneda de la compañía.

Changes:

  • Nuevo módulo completo con modelo, vista y documentación
  • Campos computados y editables para tipo de cambio informativo
  • Integración en formulario de facturas mediante herencia de vista

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 8 comments.

Show a summary per file
File Description
account_invoice_move_currency/__manifest__.py Define el módulo, dependencias y archivos de datos. Referencia archivo de seguridad inexistente
account_invoice_move_currency/__init__.py Importa el paquete models
account_invoice_move_currency/models/__init__.py Importa el modelo account_move
account_invoice_move_currency/models/account_move.py Agrega campos user_secondary_currency_id y user_exchange_rate con método compute. Tiene varios problemas de lógica
account_invoice_move_currency/views/account_move_view.xml Hereda vista de facturas para mostrar los nuevos campos. Xpath puede fallar y hay discrepancia con documentación
account_invoice_move_currency/README.rst Documentación del módulo con instalación, configuración y uso

Comment thread account_invoice_move_currency/models/account_move.py Outdated
Comment thread account_invoice_move_currency/models/account_move.py Outdated
Comment thread account_invoice_move_currency/views/account_move_view.xml Outdated
Comment thread account_invoice_move_currency/__manifest__.py Outdated
Comment thread account_invoice_move_currency/models/account_move.py Outdated
Comment thread account_invoice_move_currency/views/account_move_view.xml
Comment thread account_invoice_move_currency/models/account_move.py
Comment thread account_invoice_move_currency/models/account_move.py Outdated
@cav-adhoc cav-adhoc force-pushed the 18.0-t-63427-cav branch 2 times, most recently from 75c907d to 9d935d5 Compare February 3, 2026 18:36
This commit adds a new module 'account_invoice_move_currency' that
allows users to specify a custom exchange rate on customer invoices.
The module introduces two new fields on the account.move model:
'user_secondary_currency_id' for selecting a secondary currency,
and 'user_exchange_rate' for entering the exchange rate.
The exchange rate field is automatically computed based on the selected
secondary currency and the invoice date, but remains editable for user input.
@cav-adhoc
Copy link
Copy Markdown
Contributor Author

sigue en #258

@cav-adhoc cav-adhoc closed this Mar 3, 2026
@fw-bot-adhoc fw-bot-adhoc deleted the 18.0-t-63427-cav branch March 10, 2026 15:50
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.

3 participants