Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Slack] Add split attachments into parts when sending large changelog file #54

Merged

Conversation

RomanChetverikov
Copy link
Collaborator

@RomanChetverikov RomanChetverikov commented Oct 1, 2024

If you exceed 50 attachment objects when publishing, you get an error from the Slack server - Upload error, code=400, reason=invalid_attachments.
This crashes the whole build without the ability to save changelog.
Therefore, it is necessary to divide the number of attachments into parts of 50 each

@RomanChetverikov RomanChetverikov force-pushed the add-slack-max-attachments-count-limit branch from 19894ea to c3092a3 Compare October 3, 2024 07:54
@RomanChetverikov RomanChetverikov changed the title [Slack] Add a limit on the maximum size of attachments for publishing changelog [Slack] Add split attachments into parts when sending large changelog file Oct 3, 2024
@RomanChetverikov
Copy link
Collaborator Author

full message, with a large (exceeding 50 entries) changelog now looks like this:
image

@RomanChetverikov RomanChetverikov force-pushed the add-slack-max-attachments-count-limit branch from c3092a3 to 4932aa7 Compare October 3, 2024 08:31
… file

If you exceed 50 attachment objects when publishing, you get an error from the Slack server - Upload error, code=400, reason=invalid_attachments.
This crashes the whole build without the ability to save changelog.
Therefore, it is necessary to divide the number of attachments into parts of 50 each
@RomanChetverikov RomanChetverikov force-pushed the add-slack-max-attachments-count-limit branch from 4932aa7 to e7563c8 Compare October 3, 2024 08:49
@rinekri rinekri merged commit 0a493d4 into appKODE:main Oct 3, 2024
4 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