-
Notifications
You must be signed in to change notification settings - Fork 45
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] core: move SUPERUSER_ID #144
Conversation
18bdb7e
to
d614691
Compare
d614691
to
9a6804b
Compare
upgradeci retry with always only hr |
339fa66
to
c432edd
Compare
c432edd
to
0190f4a
Compare
0190f4a
to
cb9ff93
Compare
cb9ff93
to
d1652b6
Compare
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.
Looks good to me.
@KangOl Could you review and r+? |
upgradeci retry |
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.
@robodoo r+
@KangOl you may want to rebuild or fix this PR as it has failed CI. |
@kmagusiak @KangOl linked pull request(s) odoo/odoo#182371 not ready. Linked PRs are not staged until all of them are ready. |
In order to become a native namespace, `odoo` package must not contain any variables. This is a step in that direction. `odoo.SUPERUSER_ID` still continues to work. odoo.api.SUPERUSER_ID is often used with Environment, so we declare both in the same package. task-4069446 closes #182371 Related: odoo/upgrade-util#144 Signed-off-by: Krzysztof Magusiak (krma) <[email protected]>
In order to become a native namespace, `odoo` package must not contain any variables. This is a step in that direction. `odoo.SUPERUSER_ID` still continues to work. odoo.api.SUPERUSER_ID is often used with Environment, so we declare both in the same package. task-4069446 closes odoo#182371 Related: odoo/upgrade-util#144 Signed-off-by: Krzysztof Magusiak (krma) <[email protected]>
odoo/odoo#182371