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.

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

ResponseMetadata missing in AdminInviteRequestsResponse #1308

Closed
2 of 6 tasks
Dunkhell opened this issue May 9, 2024 · 2 comments
Closed
2 of 6 tasks

ResponseMetadata missing in AdminInviteRequestsResponse #1308

Dunkhell opened this issue May 9, 2024 · 2 comments
Assignees
Labels
bug M-T: confirmed bug report. Issues are confirmed when the reproduction steps are documented project:slack-api-client project:slack-api-client
Milestone

Comments

@Dunkhell
Copy link

Dunkhell commented May 9, 2024

  • bolt (Bolt for Java)
  • bolt-{sub modules} (Bolt for Java - optional modules)
  • slack-api-client (Slack API Clients)
  • slack-api-model (Slack API Data Models)
  • slack-api-*-kotlin-extension (Kotlin Extensions for Slack API Clients)
  • slack-app-backend (The primitive layer of Bolt for Java)

Requirements

Response for AdminInviteRequests is missing "ResponseMetadata" field

@hello-ashleyintech hello-ashleyintech self-assigned this May 9, 2024
@hello-ashleyintech hello-ashleyintech added question M-T: User needs support to use the project and removed untriaged labels May 9, 2024
@hello-ashleyintech
Copy link

hello-ashleyintech commented May 9, 2024

Hi, @Dunkhell! Thank you for submitting this! 😄

Looking through the admin.InviteRequests response here and in the other SDKs and I'm noticing ResponseMetadata is not present in any of those or other admin.invite* responses. In the API docs, there's also not really any example responses, so it's harder to tell whether this is intentional.

I'm going to check with the team to see if this is by design or not and will submit an update! 😄

@seratch seratch added bug M-T: confirmed bug report. Issues are confirmed when the reproduction steps are documented project:slack-api-client project:slack-api-client and removed question M-T: User needs support to use the project labels May 10, 2024
@seratch seratch added this to the 1.39.1 milestone May 10, 2024
@seratch
Copy link
Member

seratch commented May 10, 2024

@Dunkhell Thanks for flagging this! You're right that the response_data.next_cursor is missing in the response classes. Will resolve it quickly and ship a new patch version including the change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug M-T: confirmed bug report. Issues are confirmed when the reproduction steps are documented project:slack-api-client project:slack-api-client
Projects
None yet
Development

No branches or pull requests

3 participants