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

[ADD] add module to deliver when creating invoice #280

Closed

Conversation

laetitia-gangloff
Copy link

In sale advance payment, add option to deliver at the same time than invoice.
If picking are not validable, an error is raised.

@laetitia-gangloff laetitia-gangloff changed the title [ADD] add module to deliver when invoice [ADD] add module to deliver when creating invoice Mar 17, 2016
@laetitia-gangloff laetitia-gangloff force-pushed the 8.0-sale_invoice_auto_deliver-lga branch 2 times, most recently from abed1af to e8f2d2a Compare March 17, 2016 17:53
@coveralls
Copy link

Coverage Status

Coverage decreased (-27.0%) to 46.899% when pulling fc1cdef on acsone:8.0-sale_invoice_auto_deliver-lga into e439f5c on OCA:8.0.

@Cedric-Pigeon
Copy link
Contributor

👍 Tests and code

@rafaelbn
Copy link
Member

@laetitia-gangloff @Cedric-Pigeon this PR is failing runbot and travis, please could you review and fix errors? We are in the OCA Code Sprint and are reviewing for get merge, thanks!

@rafaelbn rafaelbn added this to the 8.0 milestone May 30, 2017
@Cedric-Pigeon
Copy link
Contributor

@rafaelbn I am in the room also ;-) I will have a look this morning.

Copy link
Member

@rafaelbn rafaelbn left a comment

Choose a reason for hiding this comment

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

Please add test, as in #280 (comment) Coverage decreased (-27.0%) to 46.899% . Thanks!

@rousseldenis
Copy link
Sponsor Contributor

Closing this as no activity. @laetitia-gangloff Please feel free to reopen it if you still need it. Take care of your PR's

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.

None yet

5 participants