PowerBIRESTAPI icon indicating copy to clipboard operation
PowerBIRESTAPI copied to clipboard

Added ReportSubscriptionsAsAdmin

Open torentlikher opened this issue 1 year ago • 1 comments

Adding a ReportSubscriptionsAsAdmin query to allow users to pull all subscriptions for the PBI Tenant.

torentlikher avatar Aug 17 '22 19:08 torentlikher

Sorry for the delay on this, i've been out on parental leave until this week. Thanks so much for contributing!

A few notes: The main problem I see with this commit is that this query will almost certainly hit API limitations in most tenants. This API has a limit of 200 per hour and by cycling through every report in the tenant, even if they don't have a subscription, it's almost certain to hit that limit in most tenants. For that reason, I'd put it in the Experimental folder just like other end points where we cannot guarantee performance. Outside of that, I'd also request that Report Subscription Users be its own separate table, and that friendly names be applied for the column names just like the other queries.

klinejordan avatar Sep 30 '22 16:09 klinejordan

Hey Jordan, Thank you for the feedback. I added a ReportSubscriptionUsersAsAdmin table. Unfortunately, I couldn't find a way to specifically identify reports with subscriptions. It seems like GetGroupsAsAdmin with reports expanded should provide that bit of information, but in my testing all subscriptions returned null. I left it as-is for now and moved both tables to Experimental. I also added friendly names to align the tables with the other queries.

torentlikher avatar Oct 16 '22 14:10 torentlikher