-
Notifications
You must be signed in to change notification settings - Fork 340
fix: update file upload to use RC 8.x rooms.media API #1070
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
base: develop
Are you sure you want to change the base?
Conversation
- Replace deprecated rooms.upload endpoint with new two-step process - Step 1: POST to rooms.media/:rid to upload file - Step 2: POST to rooms.mediaConfirm/:rid/:fileId to confirm with metadata - Fixes file upload 404 error on Rocket.Chat 8.x servers
Khizarshah01
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested locally.
The upload flow works correctly and the change to rooms.media / rooms.mediaConfirm behaves as expected.
One minor UX issue I noticed (likely unrelated to this PR): after sending an image, selecting the same file again does not trigger the upload flow. This seems related to the file input value not being reset after upload.
LGTM 👍
Screencast.From.2026-01-17.22-04-57.mp4hoping this work fine now |
Finally Checks are passing |
Khizarshah01
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
|
Hi @Spiral-Memory , Let me know if anything else needed here, whenever you get to review |
Brief Title
fix: update file upload to use RC 8.x rooms.media API
Acceptance Criteria fulfillment
[x] File uploads work on Rocket.Chat 8.x servers
[x] Thread attachments supported (tmid parameter)
[x] File description is properly sent
[x] Auth headers included in both requests
Fixes #1069
Video/Screenshots
Screencast.From.2026-01-15.21-33-44.mp4
PR Test Details
Note: The PR will be ready for live testing at https://rocketchat.github.io/EmbeddedChat/pulls/pr-<pr_number> after approval. Contributors are requested to replace
<pr_number>with the actual PR number.