Skip to content

Short-circuting the channel name to channel ID lookup.#466

Merged
elongl merged 2 commits intomasterfrom
short-circuit-slack-channels-lookup
Dec 12, 2022
Merged

Short-circuting the channel name to channel ID lookup.#466
elongl merged 2 commits intomasterfrom
short-circuit-slack-channels-lookup

Conversation

@elongl
Copy link
Contributor

@elongl elongl commented Dec 11, 2022

Solves #465.

@github-actions
Copy link
Contributor

👋 @elongl
Thank you for raising your pull request.
Please make sure to add tests and document all user-facing changes.
You can do this by editing the docs files in this pull request.

@elongl elongl linked an issue Dec 11, 2022 that may be closed by this pull request
@elongl elongl merged commit f620c8a into master Dec 12, 2022
@elongl elongl deleted the short-circuit-slack-channels-lookup branch December 12, 2022 09:47
Copy link
Contributor

@IDoneShaveIt IDoneShaveIt left a comment

Choose a reason for hiding this comment

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

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Channels iteration is too slow when sending report to Slack

2 participants