Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Get-SlackUser needs to be paginated to get all users #117

Open
simonfagerholm opened this issue May 24, 2021 · 0 comments · Fixed by simonfagerholm/PSSlack#2 · May be fixed by #118
Open

Get-SlackUser needs to be paginated to get all users #117

simonfagerholm opened this issue May 24, 2021 · 0 comments · Fixed by simonfagerholm/PSSlack#2 · May be fixed by #118

Comments

@simonfagerholm
Copy link

slack endpoint users.list is paginated and needs to be used with a cursor (if it is returned) to get all users. Currently Get-SlackUser only checks first page of results.

Implementation from Get-SlackChannel could be copied to resolve this issue, or move all pagination to Send-SlackAPI.
simonfagerholm added a commit to simonfagerholm/PSSlack that referenced this issue May 25, 2021
@simonfagerholm
Add pagination to Get-SlackUser
239522b 
Fixes: RamblingCookieMonster#117
simonfagerholm added a commit to simonfagerholm/PSSlack that referenced this issue May 25, 2021
@simonfagerholm
Add pagination to Get-SlackUser
68f186e 
Fixes: RamblingCookieMonster#117
simonfagerholm added a commit to simonfagerholm/PSSlack that referenced this issue May 25, 2021
@simonfagerholm
Add pagination to Get-SlackUser
32404c1 
Fixes: RamblingCookieMonster#117
simonfagerholm added a commit to simonfagerholm/PSSlack that referenced this issue May 25, 2021
@simonfagerholm
Add pagination to Get-SlackUser
72d4653 
Fixes: RamblingCookieMonster#117
@simonfagerholm simonfagerholm mentioned this issue May 25, 2021
Merged
simonfagerholm added a commit to simonfagerholm/PSSlack that referenced this issue May 25, 2021
@simonfagerholm
Add pagination to Get-SlackUser
1240303 
Fixes: RamblingCookieMonster#117
@simonfagerholm simonfagerholm linked a pull request May 25, 2021 that will close this issue
Open
@simonfagerholm simonfagerholm reopened this Jun 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@simonfagerholm