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] base: current company id to evaluation context #40035

Conversation

@jpp-odoo
Copy link
Contributor

jpp-odoo commented Nov 8, 2019

Add the id of the current company to the evaluation context for ir.rule
domains

@jpp-odoo jpp-odoo requested a review from nim-odoo Nov 8, 2019
@robodoo robodoo added the seen 🙂 label Nov 8, 2019
Copy link
Contributor

nim-odoo left a comment

It's more [IMP] than [ADD] (used when adding new modules). Otherwise ok

robodoo delegate+

@jpp-odoo

This comment has been minimized.

Copy link
Contributor Author

jpp-odoo commented Nov 8, 2019

indeed sorry

Add the id of the current company to the evaluation context for ir.rule
domains
@jpp-odoo jpp-odoo force-pushed the odoo-dev:13.0-opw-2093198-correct_domain_ir_rule-jpp branch from 7abb816 to ca2c2c0 Nov 8, 2019
@jpp-odoo

This comment has been minimized.

Copy link
Contributor Author

jpp-odoo commented Nov 8, 2019

robodoo r+

@robodoo robodoo added the r+ 👌 label Nov 8, 2019
@C3POdoo C3POdoo added the OE label Nov 8, 2019
@robodoo robodoo added the CI 🤖 label Nov 8, 2019
@robodoo

This comment has been minimized.

Copy link
Contributor

robodoo commented Nov 8, 2019

Linked pull request(s) odoo/enterprise#6622 not ready. Linked PRs are not staged until all of them are ready.

@robodoo robodoo closed this in 8bf4b8e Nov 13, 2019
@robodoo robodoo added merged 🎉 and removed merging 👷 labels Nov 13, 2019
@robodoo

This comment has been minimized.

Copy link
Contributor

robodoo commented Nov 13, 2019

Merged at 8bf4b8e, thanks!

@jpp-odoo jpp-odoo deleted the odoo-dev:13.0-opw-2093198-correct_domain_ir_rule-jpp branch Nov 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.