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

:arrow_up::one::two: access_restricted #113

Merged

Conversation

Projects
None yet
3 participants
@RafiZz
Copy link
Contributor

RafiZz commented Oct 26, 2018

No description provided.

@RafiZz RafiZz force-pushed the RafiZz:12.0-port-access_restricted branch 3 times, most recently from 8b58d11 to 2db1c40 Nov 1, 2018

@RafiZz RafiZz force-pushed the RafiZz:12.0-port-access_restricted branch from 2db1c40 to db56cb2 Nov 1, 2018

@@ -50,5 +50,5 @@ def fields_get(self, fields=None, **kwargs):

@api.multi
def execute(self):
res = super(ResConfigSettings, self.with_context({'access_restricted': True, 'config': self})).execute()
res = super(ResConfigSettings, self.with_context({'config': self})).execute()

This comment has been minimized.

Copy link
@yelizariev

yelizariev Nov 6, 2018

Member

What config in context is for? is it used at all?

This comment has been minimized.

@ufaks

ufaks approved these changes Nov 7, 2018

@yelizariev yelizariev merged commit fe046e0 into it-projects-llc:12.0 Nov 7, 2018

6 checks passed

Hound No violations found. Woof!
ci/branches Branch names are correct
Details
ci/runbot runbot build 23392-113-db56cb (runtime 151s)
Details
codecov/patch/backend Coverage not affected when comparing 07ce936...db56cb2
Details
codecov/patch/tests 100% of diff hit (target 100%)
Details
coverage/coveralls Coverage increased (+25.1%) to 82.848%
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.