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

Security API #51

Open
baseic opened this issue Mar 21, 2017 · 1 comment
Open

Security API #51

baseic opened this issue Mar 21, 2017 · 1 comment
Projects

Comments

@baseic
Copy link

baseic commented Mar 21, 2017

The OmniaApi.WorkWith(Ctx.Omnia()).Security(Ctx) seems to need some TLC.

E.g. IEnumerable<string> OmniaApi.WorkWith(Ctx.Omnia()).Security(Ctx) .GetCurrentUserADGroups(string[] groups);

It is not clear that groups can be an empty array, or that this only returns AD groups used in OMF.

Also, there is a need to define the difference between PermissionRoleDefinition and PermissionRole, and how to get from one to another

@khdi & @andtii

@khdi khdi added this to In Progress in Backlog Mar 24, 2017
@khdi khdi moved this from In Progress to Inbox in Backlog Mar 24, 2017
@andtii
Copy link
Contributor

andtii commented Mar 24, 2017

We need to review the Security Api

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

No branches or pull requests

2 participants