Skip to content

Support cursor pagination in search.messages API calls #2360

@gabebw

Description

@gabebw

Hi,

I notice that in the web api, search.messages is defined like this:

messages: MethodWithRequiredArgument<SearchMessagesArguments, SearchMessagesResponse>;

And SearchMessagesArguments is defined like this:

export interface SearchMessagesArguments extends TokenOverridable, TraditionalPagingEnabled, Searchable {
}

Should it extend CursorPaginationEnabled in addition to TraditionalPagingEnabled? I see that the Slack developer docs list the search.messages method as taking a cursor. But because this only extends TraditionalPagingEnabled, cursors cannot be passed to it.

Packages:

Select all that apply:

  • @slack/web-api
  • @slack/rtm-api
  • @slack/webhooks
  • @slack/oauth
  • @slack/socket-mode
  • @slack/types
  • I don't know

Requirements

Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

Metadata

Metadata

Assignees

No one assigned

    Labels

    auto-triage-skipbugM-T: A confirmed bug report. Issues are confirmed when the reproduction steps are documentedgood first issue

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions