Skip to content
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

[IMP] sale_timesheet: exclude timesheet line from billing #31043

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
4 participants
@alexey-pelykh
Copy link
Contributor

alexey-pelykh commented Feb 12, 2019

Description of the issue/feature this PR addresses:
Allow managers to exclude specific timesheet lines from billing

Current behavior before PR:
Impossible to exclude specific timesheet line from billing

Desired behavior after PR is merged:
Possible to exclude specific timesheet line from billing

This feature proves itself useful for By Task billing approach, when
a specific timesheet entry tracked towards a billable task needs to be
excluded from the Sale Order.

To exclude a specific timesheet entry from Sale Order:

  1. Go to Timesheets > Timesheet > All Timesheets
  2. Check Exclude From Billing for a specific timesheet entry

or:

  1. Go to Project > All Tasks
  2. Open specific task for editing
  3. Switch to Timesheets tab
  4. Check Exclude From Billing for a specific timesheet entry

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@robodoo robodoo added the seen 🙂 label Feb 12, 2019

@alexey-pelykh alexey-pelykh force-pushed the brainbeanapps:imp-sale_timesheet-exclude_line_from_billing branch Feb 12, 2019

@alexey-pelykh alexey-pelykh force-pushed the brainbeanapps:imp-sale_timesheet-exclude_line_from_billing branch 2 times, most recently Feb 13, 2019

@alexey-pelykh alexey-pelykh force-pushed the brainbeanapps:imp-sale_timesheet-exclude_line_from_billing branch 6 times, most recently Feb 19, 2019

@alexey-pelykh alexey-pelykh force-pushed the brainbeanapps:imp-sale_timesheet-exclude_line_from_billing branch to ede6801 Feb 22, 2019

@robodoo robodoo added the CI 🤖 label Feb 22, 2019

@alexey-pelykh

This comment has been minimized.

Copy link
Contributor Author

alexey-pelykh commented Feb 22, 2019

@jem-odoo this feature is related to #31042

@mart-e mart-e added the Services label Mar 13, 2019

@mart-e mart-e requested a review from jem-odoo Mar 13, 2019

@jem-odoo

This comment has been minimized.

Copy link
Contributor

jem-odoo commented Mar 20, 2019

AS both PR (31042 and 31043) are talking about the same thing, could you please close one of them ? Thanks.

@alexey-pelykh

This comment has been minimized.

Copy link
Contributor Author

alexey-pelykh commented Mar 20, 2019

@jem-odoo they are not the same, one allows excluding from SO at timesheet line level (#31043), other one allows excluding from SO at project task level (#31042)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.