[IMP] Timesheets: updaed time off #15815
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Collaborator
|
|
Contributor Author
| Hi @meval1006 - this is ready for a peer review whenever - no rush! |
meval1006 approved these changes Dec 22, 2025
Contributor
meval1006 left a comment
meval1006 left a comment There was a problem hiding this comment.
Hi @larm-odoo , good job on the page. Made some minor edits and suggestions.
| | ||
| Click on the magnifying glass, hovering over the concerned cell, to access all the aggregated data | ||
| on that cell (day), and see details regarding the project/task. | ||
| Once the employee has :doc:`requested time off <time_off/request_time_off>`, and the request has |
Contributor
There was a problem hiding this comment.
This doc link is wrong. Prompted an error. Could you verify with a make fast?
| needed type, and decide if you would like the requests to be validated or not. | ||
| First, activate the :ref:`developer mode <developer-mode>`, then open the **Timesheets** app, and | ||
| navigate to :menuselection:`Configuration --> Settings`. Scroll to the :guilabel:`Time Off` section | ||
| and click the :guilabel:`Time Off` checkbox. Enabling this option creates timesheet entries for |
Contributor
There was a problem hiding this comment.
Suggested change
| and click the :guilabel:`Time Off` checkbox. Enabling this option creates timesheet entries for | |
| and check or enable the :guilabel:`Time Off` checkbox. Enabling this option creates timesheet entries for |
Suggestion based on our recent style guide meeting.
| Odoo Time Off | ||
| Once enabled, two fields appear: :guilabel:`Project` and :guilabel:`Task`. The default selection | ||
| for :guilabel:`Project` is :guilabel:`Internal`, and the default selection for :guilabel:`Task` is | ||
| :guilabel:`Time Off`. These inform Odoo where to log the time off requests, which project and task. |
Contributor
There was a problem hiding this comment.
Suggested change
| :guilabel:`Time Off`. These inform Odoo where to log the time off requests, which project and task. | |
| :guilabel:`Time Off`. These inform Odoo where to log the time off requests, which project, and task. |
Edit suggestion
| respective project and task. | ||
| | On the example below, the user requested *Paid Time off* from July 13th to 15th. | ||
| .. note:: | ||
| The available options presented varies based on the installed applications. |
Contributor
There was a problem hiding this comment.
Suggested change
| The available options presented varies based on the installed applications. | |
| The available options presented vary based on the installed applications. |
| Considering that validation is not required, the requested time off is automatically displayed in | ||
| *Timesheets*. If validation is necessary, the time is automatically allocated after the responsible | ||
| person for validating does it so. | ||
| To check if a time off type required approval, check the :guilabel:`Approvals` section of the |
Contributor
There was a problem hiding this comment.
Suggested change
| To check if a time off type required approval, check the :guilabel:`Approvals` section of the | |
| To check if a time off type requires approval, check the :guilabel:`Approvals` section of the |
Comment on lines +28 to +30
| Time off requests for time off types that do **not** require validation create timesheet entries | ||
| once the employee requests time off. If a time off request requires approval, the timesheet entry | ||
| is created **after** the time off request is approved by the relevant managers. |
Contributor
There was a problem hiding this comment.
I would rewrite this section, as it is slightly confusing due to the repetition of 'time off'. Perhaps something like:
Select a time off category without validation requirements to create a timesheet record instantly. For requests requiring validation, the timesheet entry is created after the time off request is approved by the relevant managers.
| .. image:: time_off/timesheet_description.png | ||
| :align: center | ||
| :alt: View of the details of a project/task in Odoo Timeheets | ||
| To view the logged entry, open the **Timesheets** app, and navigate to the relevant time period. The |
Contributor
There was a problem hiding this comment.
Suggested change
| To view the logged entry, open the **Timesheets** app, and navigate to the relevant time period. The | |
| To view the logged entry, open the **Timesheets** app and navigate to the relevant time period. The |
| 19, 2025, the employee is home sick, and requests the day off in the **Time Off** app. | ||
| | ||
| The *time off type* they requets is *Sick Time off*, which is configured to **not** require | ||
| validaiton. The employee's time off request is automatically approved, and is logged in the |
Contributor
There was a problem hiding this comment.
Suggested change
| validaiton. The employee's time off request is automatically approved, and is logged in the | |
| validation. The employee's time off request is automatically approved, and is logged in the |
| in the **Timesheets** app under the `Internal` project, with a task of `Training`. On December | ||
| 19, 2025, the employee is home sick, and requests the day off in the **Time Off** app. | ||
| | ||
| The *time off type* they requets is *Sick Time off*, which is configured to **not** require |
Contributor
There was a problem hiding this comment.
Suggested change
| The *time off type* they requets is *Sick Time off*, which is configured to **not** require | |
| The *time off type* they request is *Sick Time off*, which is configured to **not** require |
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
Requested to update this doc on this task card.
This is the documentation task card..
Time Off requests can be automatically logged onto a Time Sheet - this was previously unclear.