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.

Sign up for GitHub

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

Jump to bottom

[9.0][ADD] sale_timesheet_description: New module #298

Closed
wants to merge 3 commits into from
Closed

[9.0][ADD] sale_timesheet_description: New module #298

wants to merge 3 commits into from

Conversation

carlosdauden
Copy link
Contributor

No description provided.

@coveralls
Copy link

coveralls commented May 8, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-0.06%) to 75.591% when pulling a20eac7 on Tecnativa:9.0-sale_timesheet_description into d368776 on OCA:9.0.

@carlosdauden carlosdauden force-pushed the 9.0-sale_timesheet_description branch 2 times, most recently from ecd05af to 53d6448 Compare May 8, 2016 18:28
@carlosdauden carlosdauden force-pushed the 9.0-sale_timesheet_description branch from 9ca634d to cbd8241 Compare May 8, 2016 22:26
@coveralls
Copy link

coveralls commented May 8, 2016 •
edited
Loading

Coverage Status

Coverage increased (+2.3%) to 77.953% when pulling cbd8241 on Tecnativa:9.0-sale_timesheet_description into d368776 on OCA:9.0.

elicoidal
elicoidal reviewed May 8, 2016
View reviewed changes
sale_timesheet_description/README.rst

To use this module, you need to:

* Go to *Sales -> Sale Orders* and create a new Sale Order.
Copy link

Choose a reason for hiding this comment

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

For procedure, better enumerated list

@elicoidal
Copy link

small details.
Other than that 👍

@coveralls
Copy link

coveralls commented May 8, 2016 •
edited
Loading

Coverage Status

Coverage increased (+2.3%) to 77.953% when pulling 1b59bc5 on Tecnativa:9.0-sale_timesheet_description into d368776 on OCA:9.0.

@coveralls
Copy link

coveralls commented May 9, 2016 •
edited
Loading

Coverage Status

Coverage increased (+2.3%) to 77.953% when pulling 53a740a on Tecnativa:9.0-sale_timesheet_description into d368776 on OCA:9.0.

@carlosdauden carlosdauden mentioned this pull request May 24, 2016
Merged
@carlosdauden carlosdauden deleted the 9.0-sale_timesheet_description branch May 27, 2016 01:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@carlosdauden @coveralls @elicoidal