markekraus / PSMSGraph

A PowerShell module for the Microsoft Graph API
MIT License
82 stars 21 forks source link

Invoke-GraphRequest should handle paging #40

Closed BatmanAMA closed 5 years ago

BatmanAMA commented 5 years ago

Paging is a well described core part of graph API and should be part of the Invoke-GraphRequest function. If you're ok with this functionality, I am willing to add it.

markekraus commented 5 years ago

I don't think Invoke-GraphRequest should handle paging. Currently, the feature functions handle it. Since this works for AAD Grapoh and MS Graph, making IGR page aware is tricky. The knowldge for the correct paging strategy lives with the feature.

BatmanAMA commented 5 years ago

Thanks for the feedback, I'll close this then.