athmangude/provisioninginfospec #344
Conversation
athmangude
requested review from
baywet,
ddyett,
MIchaelMainer,
nikithauc,
zengin and
peombwa
as code owners
athmangude
changed the title
| "isPresent": true, | ||
| "isHidden": false, | ||
| "isDisabled": false, | ||
| "id": "bed71753-0fq8-42a2-b33c-c1acb02ecd17" |
There was a problem hiding this comment.
Is the id version/cloud dependent or can we pull it up to the top level?
There was a problem hiding this comment.
It is cloud dependent not just for the whole permission but also for different schemes of the same permission
| "schemes": [ | ||
| { | ||
| "DelegatedWork": { | ||
| "isPresent": true, |
There was a problem hiding this comment.
This feels redundant. Can we implicitly assume that if we don't have an entry then it's not present?
| The "isHidden" member is a boolean value that indicates if a permission should be publicly usable in the API. | ||
|
|
||
| #### isDisabled | ||
| The "isDisabled" member is a boolean value that indicates if a permission is disabled in the environment. |
There was a problem hiding this comment.
Why do we have disabled permissions?
| This member is a JSON object that describes the provisioning info for a permission. | ||
|
|
||
| #### environments | ||
| The "environments" member is an array of objects that identifies the deployment environments in which the permission SHOULD be supported. The key of each member is the cloud environment identifier used for the [Cloud Environment Object](#cloudEnvironmentObject) |
There was a problem hiding this comment.
Does this mean that we will have a block for each of the environments and versions where the permission is supported?
| # Provisioning Info for Permissions in Graph | ||
|
|
||
| ## Abstract | ||
| The provisioning info file is a complementary JSON file to the JSON permission files that contains a list of permissions and the cloud environments in which they are present. It can serve to instruct publishing on which environments to publish a new permission to as well as show which clouds existing permissions are present in. |
There was a problem hiding this comment.
Could we merge this with the provisioning info entries already in the main file? https://github.com/microsoftgraph/msgraph-metadata/blob/dm/permissionspec/specs/permissions.md#provisioning-info-object
|
|
||
| ### <a name="versionObject"></a>Version Object | ||
| This member is a JSON object whose members are scheme objects. The key of each member is the scheme identifier used for the [Scheme Object](#schemeObject). | ||
|
|
There was a problem hiding this comment.
Looks like we should have links to Cloud Environment Object, Version Object and Scheme Object.
|
|
||
|
|
||
| ### resourceAppId | ||
| The "resourceAppId" member value provides an identifier of the resource server that is used to enforce Conditional Access checks for this permission. |
Added `resourceAppId` attribute to example
Provisioning Info for Permissions in Graph
The provisioning info file is a complementary JSON file to the JSON permission files that contains a list of permissions and the cloud environments in which they are present. It can serve to instruct publishing on which environments to publish a new permission to as well as show which clouds existing permissions are present in.