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

[IMP] add multi-company support in module sale_exceptions + faulty de… #369

Closed
wants to merge 2 commits into from

Conversation

metaminux
Copy link

…fault function signature

This commit is part of 7.0 branch for 2 years and could benefit to 8.0 too.

Not having "company_id" field in sale.exception model leads to the following error when opening or creating a sale order after migration from 7.0 to 8.0 :

ValueError: Invalid field 'company_id' in leaf "<osv.ExtendedLeaf: ('company_id', '=', False) on sale_exception (ctx: )>"

@oca-clabot
Copy link

Hey @metaminux, thank you for your Pull Request.

It looks like some users haven't signed our Contributor License Agreement, yet.
You can read and sign our full Contributor License Agreement here: http://odoo-community.org/page/website.cla
Here is a list of the users:

Appreciation of efforts,
OCA CLAbot

@oca-clabot
Copy link

Hey @metaminux,
We acknowledge that the following users have signed our Contributor License Agreement:

Appreciation of efforts,
OCA CLAbot

@rafaelbn rafaelbn added this to the 8.0 milestone Sep 26, 2019
@rousseldenis
Copy link
Sponsor Contributor

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

@metaminux
Copy link
Author

Yes, I'd like to reopen, as the only missing step was the merge... don't seem to be able to though

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

Successfully merging this pull request may close these issues.

None yet

5 participants