Skip to content

[ADD] helpdesk_mgmt_timesheet_sale#831

Open
etobella wants to merge 1 commit intoOCA:18.0from
dixmit:18.0-timesheet
Open

[ADD] helpdesk_mgmt_timesheet_sale#831
etobella wants to merge 1 commit intoOCA:18.0from
dixmit:18.0-timesheet

Conversation

@etobella
Copy link
Member

@etobella etobella commented Sep 3, 2025

Depends on #829

Allows to define sale order line directly on tickets, in a similar way that we have in PRoject tasks

@etobella etobella force-pushed the 18.0-timesheet branch 2 times, most recently from 4a5e8d0 to 6a7d1c5 Compare September 21, 2025 16:59
Copy link
Contributor

@luisDIXMIT luisDIXMIT left a comment

Choose a reason for hiding this comment

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

Code review and tested on runboat, LGTM. Only one minor comment.

domain="[('is_service', '=', True), ('order_partner_id.commercial_partner_id.id', 'parent_of', parent.partner_id), ('is_expense', '=', False), ('state', '=', 'sale'), ('is_downpayment', '=', False)]"
optional="hide"
/>
<field
Copy link
Contributor

Choose a reason for hiding this comment

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

It’s the same field with the same configuration, but one is visible to salespeople and the other to the rest of the groups. Is it correct?

Copy link
Member Author

Choose a reason for hiding this comment

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

probably not 😆

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.

2 participants