serokell / tzbot

Timezone bot for Slack
Mozilla Public License 2.0
7 stars 2 forks source link

[#7] Cache slack data #25

Closed YuriRomanowski closed 1 year ago

YuriRomanowski commented 1 year ago

Description

Problem: The bot uses two methods, getUser and getChannelMembers, which are subjects to Slack's rate limits, and we need to avoid unneeded calls to those methods.

Solution: Add caching for user info and channel members.

Related issue(s)

Fixed #7

:white_check_mark: Checklist for your Pull Request

Related changes (conditional)

Stylistic guide (mandatory)

YuriRomanowski commented 1 year ago

Guys how do you think, do we need some tests here for the actual events processing and caching?

alexd1971 commented 1 year ago

Probably it would be better to rebase this branch onto main before review.