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

10.0-BR001143-T23294-timesheet_activity_report #191

Open
wants to merge 4 commits into
base: 10.0
Choose a base branch
from

Conversation

nikul-serpentcs
Copy link

Timesheet Activity Report

This module extends the functionality of project adding a timesheet report listed at timesheet completion analysis.
You can use this report to communicate the timesheet analysis to the project members.

@sudhir-serpentcs sudhir-serpentcs mentioned this pull request Sep 6, 2017
11 tasks
@nikul-serpentcs
Copy link
Author

@elicoidal Could you please review code.

#175 #313

@victormmtorres victormmtorres changed the title [WIP] [10.0] [BR001143] [T23294] timesheet_activity_report 10.0-WIP-BR001143-T23294-timesheet_activity_report Sep 6, 2017
Copy link
Contributor

@elicoidal elicoidal left a comment

Choose a reason for hiding this comment

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

Copyright.
Other than that needs to be tested.
@seb-elico

@@ -1,5 +1,5 @@
# -*- coding: utf-8 -*-
# © 2015 Elico corp (www.elico-corp.com)
# Copyright 2015-17 Elico corp (www.elico-corp.com)
Copy link
Contributor

Choose a reason for hiding this comment

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

Copyright 2015-2017 Elico corp (https://www.elico-corp.com)

Copy link
Contributor

Choose a reason for hiding this comment

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

(everywhere)

@elicoidal elicoidal changed the title 10.0-WIP-BR001143-T23294-timesheet_activity_report 10.0-BR001143-T23294-timesheet_activity_report Sep 6, 2017
@elicoidal elicoidal added this to the 10.0 milestone Sep 6, 2017
@elicoidal
Copy link
Contributor

@victormartinelicocorp

@sudhir-serpentcs
Copy link

Travis is failed due to dependency issue.
@elicoidal @victormartinelicocorp

'Project',
readonly=True
)
# project_state = fields.Char(
Copy link
Contributor

Choose a reason for hiding this comment

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

@nikul-serpentcs remove dead code

Copy link
Author

Choose a reason for hiding this comment

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

@victormartinelicocorp Removed dead code, Please review it.

@iceyxu
Copy link

iceyxu commented Sep 26, 2017

@Elico-Corp/odoo-reviewers what status for this PR?

@elicoidal
Copy link
Contributor

@iceyxu We need to tests all these features but runbot is not yet ready.
I would suggest that @victormartinelicocorp prepare the trunk environment for Elico with latest modifications and that you test them

@nikul-serpentcs
Copy link
Author

@elicoidal If no need then Close.

@elicoidal
Copy link
Contributor

Hi @nikul-serpentcs we will keep it for the time being but will cleanse everything in few weeks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants