ResponseMetadata missing in AdminInviteRequestsResponse

slackapi / java-slack-sdk

Slack Developer Kit (including Bolt for Java) for any JVM language

https://slack.dev/java-slack-sdk/

MIT License

568 stars 210 forks source link

ResponseMetadata missing in AdminInviteRequestsResponse #1308

Closed Dunkhell closed 2 months ago

Dunkhell commented 2 months ago

[ ] bolt (Bolt for Java)
[ ] bolt-{sub modules} (Bolt for Java - optional modules)
[x] slack-api-client (Slack API Clients)
[x] 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 commented 2 months ago

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 commented 2 months ago

@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.