Skip to content

Updated AzureActivity.json - #796

Merged
Shain (shainw) merged 2 commits into
Azure:masterfrom
samikroy:master
Sep 24, 2020
Merged

Updated AzureActivity.json#796
Shain (shainw) merged 2 commits into
Azure:masterfrom
samikroy:master

Conversation

@samikroy

@samikroy Samik Roy (samikroy) commented Jun 26, 2020

Copy link
Copy Markdown
Contributor

Reason : Making Caller pill similar to ResourceGroup to get data from Log Analytics workspace instead of resource group.This will be enable users with limited permission only to workspace to deploy the workbook, instead the resource group.

image

Reason : Making Caller pill similar to ResourceGroup to get data from Log Analytics workspace instead of resource group.This will be enable users with limited permission only to workspace to deploy the workbook, instead the resource group.
Comment thread Workbooks/AzureActivity.json
Updated workbooksMetadata for AzureActivity workbook fix for making Caller pill similar to ResourceGroup to get data from Log Analytics workspace instead of resource group.This will be enable users with limited permission only to workspace to deploy the workbook, instead the resource group..
@samikroy

Copy link
Copy Markdown
Contributor Author

Amit Bergman (@Amitbergman) Could you please help with the review,

@samikroy

Copy link
Copy Markdown
Contributor Author

Shain (@shainw) - Could you please help with review.

@shainw

Copy link
Copy Markdown
Contributor

Shain (@shainw) - Could you please help with review.

I pinged Amit Bergman (@Amitbergman) to Approve and then we can complete this one.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me

@shainw
Shain (shainw) merged commit ccae1fb into Azure:master Sep 24, 2020
@samikroy

Copy link
Copy Markdown
Contributor Author

Thank you Amit Bergman (@Amitbergman) & Shain (@shainw) for your help here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants