Skip to content

Add missing API methods#1222

Merged
seratch merged 2 commits intoslackapi:mainfrom
seratch:21-04-30-methods
May 7, 2021
Merged

Add missing API methods#1222
seratch merged 2 commits intoslackapi:mainfrom
seratch:21-04-30-methods

Conversation

@seratch
Copy link
Copy Markdown
Contributor

@seratch seratch commented May 1, 2021

Summary

This pull request adds the following API methods to WebClient.

  • admin.apps.clearResolution
  • admin.apps.uninstall

Also, I moved the non-conversation API methods to the bottom as they no longer work (we may want to delete them in the near future but we won't do so in v6.2).

Requirements (place an x in each [ ])

@seratch seratch added semver:minor enhancement M-T: A feature request for new functionality pkg:web-api applies to `@slack/web-api` labels May 1, 2021
@seratch seratch added this to the web-api@6.2 milestone May 1, 2021
public abstract apiCall(method: string, options?: WebAPICallOptions): Promise<WebAPICallResult>;

public readonly admin = {
// TODO: admin.analytics.getFile
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

As described at #1191 , this endpoint returns binary data. We will work on it separately.

list:
bindApiCall<AdminAppsRestrictedListArguments, AdminAppsRestrictedListResponse>(this, 'admin.apps.restricted.list'),
},
// TODO: Use AdminAppsUninstallResponse when it's available
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

AdminAppsUninstallResponse will be available once we merge slackapi/java-slack-sdk#735

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is done.

},
};

public readonly channels = {
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

},
};

public readonly views = {
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Moved this one to make the list here alphabetical order

is_pending_ext_shared?: boolean;
connected_team_ids?: string[];
conversation_host_id?: string;
channel_email_addresses?: string[];
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ran ./scripts/generate-web-api-types.sh

@seratch seratch requested review from mwbrooks and stevengill May 1, 2021 00:19
@seratch seratch merged commit a950443 into slackapi:main May 7, 2021
@seratch seratch deleted the 21-04-30-methods branch May 7, 2021 00:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

2 participants