-
Notifications
You must be signed in to change notification settings - Fork 843
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
Fix next_cursor Flag in SlackResponse #1407
Comments
Hello @lre12 thanks for writing in 💯 Contributions to the project are always greatly appreciated 🥇 take a look at our creating-a-pull-request docs for information on this Before proceeding would you be able to outline which APIs return |
@lre12 Could you share a bit more details about |
I am using the Slack enterprise called example code)
The token and cookie were obtained from the enterprise admin site. |
@lre12 Thanks for your response. Now I understand your point. You're attempting to use this SDK to call an endpoint that is not available for 3rd party apps. (Indeed, you can call We're willing to make the code even more robust for potential future changes on the public platform, but we do not support your specific use case with private/hidden APIs. We would appreciate it if you could understand this. |
Ah, I understand. Thank you. |
In the SlackResponse, the presence of the next_cursor is determined by the
_next_cursor_is_present
method.In most APIs, the next_cursor is returned as an empty string ("") as seen in
_next_cursor_is_present
.However, in some APIs, the next_cursor is returned as null. To handle this situation, I would like to make modifications to the
_next_cursor_is_present
method.example) slackbot.responses.list
Category (place an
x
in each of the[ ]
)Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.
The text was updated successfully, but these errors were encountered: