Skip to content

Conversation

@kgopal492
Copy link
Collaborator

SUMMARY

  • Slack logic for conversations_list API does not handle rate limiting - update to add sleep & retries when we get rate limit errors
  • Cache slack channels result
  • Some logic errors in Slack channels for file_upload_v2 (strip leading # in channel name, add only channel ids to new target)
  • Saving original target string from original recipient config to be able to migrate IM channels to v2 in future (IMs won't be returned in conversations_list)

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

  • Has associated issue:
  • Required feature flags:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

@kgopal492 kgopal492 merged commit 4f25ce4 into pinterest:pinterest-release-4.1.1 Nov 16, 2025
12 of 27 checks passed
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.

2 participants