Skip to content

[bugfix] multiple files uploaded to slack as different messages#116

Merged
cramforce merged 1 commit intomainfrom
bugfix/multiple-attachments
Feb 26, 2026
Merged

[bugfix] multiple files uploaded to slack as different messages#116
cramforce merged 1 commit intomainfrom
bugfix/multiple-attachments

Conversation

@visyat
Copy link
Collaborator

@visyat visyat commented Feb 26, 2026

When multiple files posted in the same message thread.post(text, files: [ ... ]);, Slack adapter posts as individual messages, rather than one message with multiple attachments.

@vercel
Copy link
Contributor

vercel bot commented Feb 26, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
chat Ignored Ignored Open in v0 Feb 26, 2026 0:52am

@github-actions
Copy link
Contributor

Preview Branch Testing

To test this PR with real webhook traffic:

  1. Go to /settings on the production deployment
  2. Enter this PR's Vercel preview URL
  3. Save — all webhook requests will now be proxied to this preview

To disable: Clear the URL on the settings page.

Copy link
Contributor

@vercel vercel bot left a comment

Choose a reason for hiding this comment

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

Additional Suggestion:

Mock return value for files.uploadV2 uses a flat structure that doesn't match the nested response structure the adapter code expects, causing file IDs to always resolve to an empty array.

Fix on Vercel

@cramforce cramforce merged commit 2623a50 into main Feb 26, 2026
8 checks passed
@cramforce cramforce deleted the bugfix/multiple-attachments branch February 26, 2026 00:58
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