Skip to content

Commit a935a88

Browse files
Merge branch 'master' into patch-2
2 parents 94cea4f + a668a98 commit a935a88

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

.github/workflows/slack/README.md

+4
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,10 @@ Three main resources are created using the slack integration:
6565
- Use of bot token of the format `xoxo-<your-slack-token>` is not supported for creating user groups.
6666
- The user group should be unique across the workspace (i.e., no channel, user, or user group should have the same handle).
6767
- Please [import](#importing-existing-resources) the user groups to terraform state if they already exist in the workspace, as they **cannot be deleted** in Slack 😢.
68+
- **You have to invite the bot to all managed channels to make it work.**
69+
70+
> [!WARNING]
71+
> **Creation of multitudes of groups leads to a lot of API calls and can lead to rate limiting. Please be cautious while creating groups, otherwise you will get `Error: couldn't get usergroups: slack rate limit exceeded, retry after 30s`**. Related issue (not fixed even thought it is closed): https://github.com/pablovarela/terraform-provider-slack/issues/64
6872
6973
> [!IMPORTANT]
7074
> The terraform state will overwrite any description, name, or topic change. It is better to manage the changes in the YAML files and then apply them. However, the terraform state will not affect bookmarks, pinned items, etc.

.github/workflows/slack/groups/groups.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222

2323
- name: "Studio"
2424
handle: "studio"
25-
description: "To notify all regular Studio contributors at once. If you don't contribute regularly but want to get notified too, please open a PR to get added."
25+
description: "To notify all regular Studio contributors at once, please open PR to get added"
2626
users:
2727
- "U01RWDD69PZ"
2828
- "U0572R8J927"

0 commit comments

Comments
 (0)