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

[9.0] Add sale_payment #403

Closed
wants to merge 2 commits into from
Closed

Conversation

florian-dacosta
Copy link
Contributor

This module aims to replace modules sale_payment_method and sale_quick_payment from v8.

It allows you to to pay add payment on draft sale order, it allows also to link payment to sale order.
It can be a usefull information for user to see if sale order has associated payment and could also be used for with sale_exceptions for example, to create validation rule if order is paid, etc...

@mourad-ehm
Copy link
Contributor

When I "Register a payment" from sale order then I create an invoice, This one is not passed to state "Paid" automatically. Why this one remain in "Open" state? Is a normal (wished) behaviour ?

@mourad-ehm
Copy link
Contributor

runbot is red, but there no error message. What's happen ?
Some one has a response ?

@florian-dacosta
Copy link
Contributor Author

@mourad-ehm
I don't think the feature was existing in previous versions and I did not need it for my customer as we use the mass_reconcile module for all reconciliation.
Of course, I guess it would be a nice feature to have and could be added to the module in the future, if anyone need this.
Regarding the runbot, I do not know...

@LoisRForgeFlow
Copy link
Contributor

Seem similar problem to what happens here: #407

In your case is must_have_dates but also type and uom_id

@rafaelbn rafaelbn added this to the 9.0 milestone May 30, 2017
@florian-dacosta florian-dacosta mentioned this pull request Mar 29, 2018
50 tasks
@rousseldenis
Copy link
Sponsor Contributor

@florian-dacosta Is this still relevant? As no activity, it will be closed soon.

@florian-dacosta
Copy link
Contributor Author

@rousseldenis
Well, actually, I think this module is still relevant, but v9 is old already and this PR probably won't ever move forward, so, agree to close it.

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