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

feat (web-api): add support for teams.externalTeams.disconnect API #1837

Merged
merged 1 commit into from
Jul 3, 2024

Conversation

filmaj
Copy link
Contributor

@filmaj filmaj commented Jul 3, 2024

No description provided.

@filmaj filmaj added semver:minor enhancement M-T: A feature request for new functionality area:typescript issues that specifically impact using the package from typescript projects pkg:web-api applies to `@slack/web-api` labels Jul 3, 2024
@filmaj filmaj added this to the [email protected] milestone Jul 3, 2024
@filmaj filmaj requested a review from a team July 3, 2024 15:56
@filmaj filmaj self-assigned this Jul 3, 2024
Copy link

codecov bot commented Jul 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.85%. Comparing base (4ef80b7) to head (e506a42).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1837      +/-   ##
==========================================
+ Coverage   81.83%   81.85%   +0.01%     
==========================================
  Files          35       35              
  Lines        7774     7781       +7     
  Branches      318      318              
==========================================
+ Hits         6362     6369       +7     
  Misses       1400     1400              
  Partials       12       12              
Flag Coverage Δ
cli-hooks 94.94% <ø> (ø)
cli-test 54.16% <ø> (ø)
oauth 76.53% <ø> (ø)
socket-mode 59.59% <ø> (ø)
web-api 96.57% <100.00%> (+<0.01%) ⬆️
webhook 95.27% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

/**
* @description Returns a list of all the external teams connected and details about the connection.
* @see {@link https://api.slack.com/methods/team.externalTeams.list `team.externalTeams.list` API reference}.
*/
lookup: bindApiCall<TeamExternalTeamsListArguments, TeamExternalTeamsListResponse>(this, 'team.externalTeams.list'),
list: bindApiCall<TeamExternalTeamsListArguments, TeamExternalTeamsListResponse>(this, 'team.externalTeams.list'),
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made a booboo here previously 😞 😢

@@ -37,10 +38,6 @@ interface Message extends Channel {
/** @description Unique identifier of message. */
ts: string;
}
interface TargetTeam {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Factored this out into common.js as it's also used for various team.* APIs.

@filmaj filmaj merged commit a18c1ea into main Jul 3, 2024
30 checks passed
@filmaj filmaj deleted the teams-disconnect branch July 3, 2024 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:typescript issues that specifically impact using the package from typescript projects enhancement M-T: A feature request for new functionality pkg:web-api applies to `@slack/web-api` semver:minor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant