AdminAppsRequestsListRequest class missing `enterpriseId` field

[andrewtcobb]

The admin.apps.request.list API method has an optional argument enterprise_id that is not present in the corresponding AdminAppsRequestsListRequest class in the SDK. This argument is present for both the AdminAppsApproveRequest and the AdminAppsRestrictedListRequest classes, but missing from AdminAppsRequestsListRequest. Without this missing attribute, calls to the admin.apps.request.list API endpoint cannot be made with the SDK in situations where the enterprise_id is required instead of the team_id.

Reproducible in:

The Slack SDK version

v1.19.0

Java Runtime version

openjdk 17.0.1 2021-10-19
OpenJDK Runtime Environment (build 17.0.1+12-39)
OpenJDK 64-Bit Server VM (build 17.0.1+12-39, mixed mode, sharing)

OS info

ProductName: Mac OS X
ProductVersion: 10.15.7
BuildVersion: 19H1519
Darwin Kernel Version 19.6.0: Tue Oct 12 18:34:05 PDT 2021; root:xnu-6153.141.43~1/RELEASE_X86_64

Steps to reproduce:

1. Use v1.19.0 or lower of the Java Slack API Client
2. Attempt to get or set the value of enterpriseId on the AdminAppsRequestsListRequest class

Expected result:

The value of enterpriseId can be read or written to.

Actual result:

The attribute enterpriseId does not exist on the class.

[mwbrooks]

Hey @andrewtcobb, thanks a lot for pointing out that the enterprise_id field is missing from AdminAppsRequestListRequest. 🙇🏻

We're thinking that this parameter was added afterwards and is probably missing for the Python and Node.js SDKs as well!

I've labelled this issue as an enhancement and assigned it to our next milestone (it may be triaged to another release). Once confirmed, we'll need to also create issues for the Python/Node.js SDKs.

Appreciate that you took the time to submit this issue! 👍🏻