-
-
Notifications
You must be signed in to change notification settings - Fork 348
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
[16.0][ADD] sale_timesheet_timeline: Dates planning in sales order lines #708
[16.0][ADD] sale_timesheet_timeline: Dates planning in sales order lines #708
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool. Tested 👍
f05b170
to
fe78817
Compare
I have added a new view that extends the order view of the portal to show a separate table with the data of the lines that have a planning set up. This is shown in a new section in the view called ‘Planning’. @chienandalu If you want you can check again |
ed17e79
to
eb71e77
Compare
14e5f80
to
660a452
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just a few comments
<field | ||
name="task_date_start" | ||
optional="hide" | ||
attrs="{'invisible': [('product_type', '!=', 'service')]}" | ||
/> | ||
<field | ||
name="task_date_end" | ||
optional="hide" | ||
attrs="{'invisible': [('product_type', '!=', 'service')]}" | ||
/> | ||
<field | ||
name="task_user_ids" | ||
optional="hide" | ||
widget="many2many_avatar_user" | ||
attrs="{'invisible': [('product_type', '!=', 'service')]}" | ||
/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what about this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ping @pedrobaeza
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This comment was marked as resolved.
This comment was marked as resolved.
7dd120c
to
55a1261
Compare
This module allows to plan in advance the start/end dates and assignees for the sales order lines that are services, and they are transferred to the created task (if any) when confirming the order. It includes a timeline view for pre-viewing and managing the planning. TT50569
55a1261
to
d89d672
Compare
This PR has the |
/ocabot merge nobump |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at 53760a9. Thanks a lot for contributing to OCA. ❤️ |
This module allows to plan in advance the start/end dates and assignees for the sales order lines that are services, and they are transferred to the created task (if any) when confirming the order.
It includes a timeline view for pre-viewing and managing the planning.
@Tecnativa TT50569