search

slackapi / java-slack-sdk

Slack Developer Kit (including Bolt for Java) for any JVM language
https://slack.dev/java-slack-sdk/
MIT License
573 stars 214 forks source link

requestSharedInvite approve/deny/list sample JSON payloads #1333

Closed filmaj closed 2 weeks ago

filmaj commented 3 months ago

These are samples for the following APIs (only available in dev right now):

I am issuing this PR to the slack-connect-invite-automation branch as a base, as these APIs are being piloted and should not be merged into main until after the customer pilot for these APIs is complete.

codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 71.06918% with 46 lines in your changes missing coverage. Please review.

Project coverage is 74.81%. Comparing base (d8eb0d2) to head (9f8f1d2).

Files with missing lines Patch % Lines
...ck/api/bolt/jakarta_socket_mode/SocketModeApp.java 62.50% 28 Missing and 5 partials :warning:
bolt/src/main/java/com/slack/api/bolt/App.java 57.14% 2 Missing and 4 partials :warning:
...m/slack/api/bolt/request/builtin/EventRequest.java 57.14% 0 Missing and 3 partials :warning:
...ck/api/bolt/request/builtin/ViewClosedRequest.java 33.33% 1 Missing and 1 partial :warning:
.../main/java/com/slack/api/bolt/context/Context.java 83.33% 0 Missing and 1 partial :warning:
...k/api/bolt/request/builtin/BlockActionRequest.java 85.71% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## slack-connect-invite-automation #1333 +/- ## ===================================================================== - Coverage 74.95% 74.81% -0.14% - Complexity 4186 4273 +87 ===================================================================== Files 451 457 +6 Lines 12930 13276 +346 Branches 1331 1369 +38 ===================================================================== + Hits 9692 9933 +241 - Misses 2466 2545 +79 - Partials 772 798 +26 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

seratch commented 3 months ago

@filmaj If you just want to generate the response type, you can manually add it for now. Once the response data structure is finalized and this repo's main branch supports it, the manually crafted one will be automatically replaced by the quicktype script then.