-
Notifications
You must be signed in to change notification settings - Fork 157
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
Get-VSTeamUser return only 500 users event if there is more, Filtering by Get-VSTeamUser | ? DisplayName also not finding existing user #412
Comments
@JanuszNowak hard to test for me. Any idea how I can invite more than 500 users on my own private Org to test this case? |
Please refer to https://docs.microsoft.com/en-us/rest/api/azure/devops/graph/users/list?view=azure-devops-rest-6.1 As suggestion, the Get-VSTeamUser should have a -continuationToken parameter to allow sucesive calls or manage internally the continuation token and return the complete list of users Also as additional note: the API in the v6.0 supports an aditional parameter scope that allows to filter users by project, collection... given the descriptor |
@SebastianSchuetze, @DarqueWarrior I would like to open a discussion here: This is a problem because Invoke-RestMethod in powershell Desktop (5.1) does not support manage headers in the response. This support was introduced later in powershell 6.0 So, I see some options:
|
@mnieto I cannot speak for @DarqueWarrior but I have his trust. We already decided to ditch the support for all TFS versions and tend to only support Azure DevOps versions that Microsoft supports. There is already a branch that will introduce the removal of these API endpoint (you can still control ist by enforcing it if needed). But we could introduce this also in Version 8.0.0 and also only support pwsh 6+. People who can't use it are free to use version less than 8.0.0. Having said that: We will not create a new _callAPI but rather change the existing. Do you want to give it a shot? |
Ok, I can try to work on that. which branch should I take as a starting point? feature/v8.0.0? or trunk, as usual? My idea to implement this is just to add an optional parameter To manage |
Make a branch based on V8.0.0. And go from there. I will update that branch with the newest changes. The PR would also be into V8.0.0 And feel free to also integrate that into the continuation token. Could you also create an extra MD file to explain how and when to use the internal function for development? In that way new devs and also we can use it in all cmdlets where it is needed. |
@SebastianSchuetze I would probably go with rest api https://learn.microsoft.com/en-us/rest/api/azure/devops/memberentitlementmanagement/user-entitlements/add?view=azure-devops-rest-6.0&tabs=HTTP |
You wanna give it a try @JanuszNowak ? |
Steps to reproduce
Expected behavior
Actual behavior
Environment data
OS
Server
The text was updated successfully, but these errors were encountered: