-
-
Notifications
You must be signed in to change notification settings - Fork 176
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
Feature/account fiscal position rule purchase migration 8.0 #11
Feature/account fiscal position rule purchase migration 8.0 #11
Conversation
858289c
to
e8ac579
Compare
ff2b2ac
to
6ea90bf
Compare
just rebased on 8.0 so tests are passing again. Need review now. |
👍 |
6ea90bf
to
329f931
Compare
ping @mileo @danimaribeiro time for a review? |
LGTM 👍 |
<field name="arch" type="xml"> | ||
<field name="partner_id" position="attributes" > | ||
<attribute name="context_fiscalrules">{'company_id': company_id}</attribute> | ||
</field> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
web_context_tunnel is needed in the depends.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
You could add view and model folders, and remove the file account_fiscal_position_rule_purchase_security.xml which is empty. The rules are working as expected after adding web_context_tunnel as dependency. |
329f931
to
4de9032
Compare
5b14e24
to
fc9b5f8
Compare
@danimaribeiro took your considerations into account. Good catch about the missing web_context_tunnel dependency. So merging now as the module was approved otherwise. |
…rule_purchase_migration_8.0 Feature/account fiscal position rule purchase migration 8.0
No description provided.