Fixes an issue with slack integration, where newly created channels were not being displayed in the list of available channels.
It was because, if we do not pass the limit argument, slack API always returns 100 results at max, but in our scenarios, user had more than 100 channels ! Now a limit is set to 200 (recommended by slack) and we additionally use next_cursor .next_cursor is a pagination token returned by the Slack API. It indicates the presence of additional pages of data. When next_cursor is not empty, it should be included in subsequent requests to fetch the next page of data.
Also fixes incorrect Redirect url in docs
How should this be tested?
Test with a large number of channels or, decrease the pagination limit
Checklist
Required
[x] Filled out the "How to test" section in this PR
What does this PR do?
Fixes an issue with slack integration, where newly created channels were not being displayed in the list of available channels.
It was because, if we do not pass the limit argument, slack API always returns 100 results at max, but in our scenarios, user had more than 100 channels ! Now a limit is set to 200 (recommended by slack) and we additionally use
next_cursor
.next_cursor
is a pagination token returned by the Slack API. It indicates the presence of additional pages of data. Whennext_cursor
is not empty, it should be included in subsequent requests to fetch the next page of data.Also fixes incorrect Redirect url in docs
How should this be tested?
Test with a large number of channels or, decrease the pagination limit
Checklist
Required
pnpm build
console.logs
git pull origin main
Appreciated