Skip to content

[IMP] account, sale, l10n_it_edi, l10n_tr_nilvera_einvoice: more visibility about the sent status of invoice #4811

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

Open
wants to merge 1 commit into
base: 18.0-rd-accounting-onboarding-malb
Choose a base branch
from

Conversation

abmn-odoo
Copy link

[IMP] account, sale, l10n_it_edi, l10n_tr_nilvera_einvoice: more visibility about the sent status of invoice

With the evolution of the electronic invoicing, but also in general, it's a good idea to show more easily when an invoice is sent or not by the send & print.

User can easily see the difference between invoices which are Sent or not.

is_move_sent is now ready to be removed as it is replaced by move_sent_state.

task-4908889

…bility about the sent status of invoice

With the evolution of the electronic invoicing, but also in general, it's a good idea to show more easily when an invoice is sent or not by the send & print.

User can easily see the difference between invoices which are Sent or not.

`is_move_sent` is now ready to be removed as it is replaced by `move_sent_state`.

task-4908889
@abmn-odoo abmn-odoo requested a review from malb-odoo July 3, 2025 09:40
@abmn-odoo abmn-odoo self-assigned this Jul 3, 2025
@robodoo
Copy link

robodoo commented Jul 3, 2025

This PR targets the un-managed branch odoo-dev/odoo:18.0-rd-accounting-onboarding-malb, it needs to be retargeted before it can be merged.

Copy link

@malb-odoo malb-odoo left a comment

Choose a reason for hiding this comment

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

The spec was maybe not that clear, small change to do

copy=False,
readonly=True,
help="It indicates that the invoice/payment has been sent or the PDF has been generated.",
)

Choose a reason for hiding this comment

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

Maybe the spec was not clear enough we actually don't want to get rid of is_move_sent but use it 👀 With the is_move_sent you can do compute on this field, it will reduce the diff a looot you will see ahah

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