Skip to content

Conversation

@vicire1
Copy link

@vicire1 vicire1 commented Nov 4, 2025

When an invoice is sent there is no other way to ee it than in the form view by the button send & print thus I added an optional hide field in the list view that shows the send status and made 2 filters sent and not sent to easily differentiate those invoices

@robodoo
Copy link

robodoo commented Nov 4, 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.

Commit message needs to be improved 👀

send_status = fields.Selection(
selection=[
('sent', "Sent"),
('sending', "Sending..."),

Choose a reason for hiding this comment

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

hmmm why ?

Comment on lines 801 to 804
if move.is_move_sent:
move.send_status = 'sent'
elif move.is_move_sent:
move.send_status = 'sending'

Choose a reason for hiding this comment

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

seems :suspect:

for move in self:
move.is_being_sent = bool(move.sending_data)

@api.depends('is_being_sent', 'is_move_sent')

Choose a reason for hiding this comment

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

This is enough 😄 We don't care about the move being sent asynchronously 😄

Suggested change
@api.depends('is_being_sent', 'is_move_sent')
@api.depends('is_move_sent')

@vicire1 vicire1 force-pushed the 18.0-visibility-sent-invoice-videc branch 3 times, most recently from bcd626e to fce748e Compare November 5, 2025 16:14
@vicire1 vicire1 changed the title [IMP] Account: adding send status column and filter [IMP] account: improved send status of invoices visibility Nov 5, 2025
When an invoice is sent there is no other way to ee it than in the form view by the button send & print thus I added an optional hide field in the list view that shows the send status and made 2 filters sent and not sent to easily differentiate those invoices

task-5231328
@vicire1 vicire1 force-pushed the 18.0-visibility-sent-invoice-videc branch from fce748e to 473f5d9 Compare November 5, 2025 16:17
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.

4 participants