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

Dont throw an error notification in case no Bpartner bill-to address exists, but only show an error message over the field #5843

Merged
merged 4 commits into from Nov 28, 2019

Conversation

@TheBestPessimist
Copy link
Member

TheBestPessimist commented Nov 27, 2019

In this way the bpartner field is no longer cleared (when the transaction is rolled back by the thrown exception)

#5834

 Sales Order recording without Billto Partner #3209
…exists, but only show an error message over the field.

In this way the bpartner field is no longer cleared (when the transaction is rolled back by the thrown exception)

#5834
…instead of blindly taking those from bpartner (which may not be "invoice locations"

#5834
@metas-ts metas-ts merged commit 57917e5 into master Nov 28, 2019
2 checks passed
2 checks passed
Codacy/PR Quality Review Up to standards. A positive pull request.
Details
continuous-integration/jenkins/branch This commit looks good
Details
@metas-ts metas-ts deleted the gh5834 branch Nov 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.