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

[FIX] hr_timesheet: no leading 0 in report total #38300

Conversation

@Florimond
Copy link
Contributor

commented Oct 9, 2019

  • Install timesheets and studio.
  • In timesheets add a time of 0.5 ( half an hour).
  • Enter studio
  • Switch to the Reports tab, and click Timesheet Entries.

Before this commit:

The total time is displayed as 0:30.

After this commit:

The report_timesheet_template view has been modified to display the total
time as 00:30.

Description of the issue/feature this PR addresses:

Current behavior before PR:

Desired behavior after PR is merged:

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

- Install timesheets and studio.
- In timesheets add a time of 0.5 ( half an hour).
- Enter studio
- Switch to the Reports tab, and click Timesheet Entries.

Before this commit:

The total time is displayed as 0:30.

After this commit:

The `report_timesheet_template` view has been modified to display the total
time as 00:30.
@Florimond

This comment has been minimized.

Copy link
Contributor Author

commented Oct 9, 2019

@robodoo robodoo added the seen 🙂 label Oct 9, 2019
Copy link
Contributor

left a comment

robodoo r+

robodoo pushed a commit that referenced this pull request Oct 9, 2019
- Install timesheets and studio.
- In timesheets add a time of 0.5 ( half an hour).
- Enter studio
- Switch to the Reports tab, and click Timesheet Entries.

Before this commit:

The total time is displayed as 0:30.

After this commit:

The `report_timesheet_template` view has been modified to display the total
time as 00:30.

closes #38300

Signed-off-by: Nicolas Martinelli (nim) <nim@odoo.com>
@robodoo robodoo added the merging 👷 label Oct 9, 2019
@C3POdoo C3POdoo added the RD label Oct 9, 2019
@robodoo robodoo added merged 🎉 and removed merging 👷 labels Oct 9, 2019
@robodoo

This comment has been minimized.

Copy link
Contributor

commented Oct 9, 2019

Merged at 3ae45fc, thanks!

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