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.

Sign up for GitHub

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

Jump to bottom

Local echo for file upload. #1288

Closed
ara4n opened this issue Jul 8, 2023 · 3 comments
Closed

Local echo for file upload. #1288

ara4n opened this issue Jul 8, 2023 · 3 comments
Labels
A-File-Upload Attachments and file uploads A-Local echo O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements X-Needs-Backend X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue

Comments

@ara4n
Copy link
Member

ara4n commented Jul 8, 2023

Steps to reproduce

  1. Upload an image
  2. Nothing appears in the timelime until remote echo of the event has bounced off the server.

Outcome

What did you expect?

Local echo of the thing you're trying to send, both while you're sending it (see #1283) and after it's sent.

What happened instead?

Upload appears like it's fallen into the void (especially if server is slow, overloaded, or bad network) - or, alternatively, suddenly unexpectedly pops the timeline a second or so later once the remote echo comes in.

Your phone model

No response

Operating system version

No response

Application version

279

Homeserver

No response

Will you send logs?

No
@ara4n ara4n added the T-Defect label Jul 8, 2023
@kittykat kittykat added A-Local echo S-Major Severely degrades major functionality or product features, with no satisfactory workaround O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience A-File-Upload Attachments and file uploads Z-Schedule labels Jul 12, 2023
@pixlwave pixlwave added X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements and removed T-Defect labels Jul 19, 2023
@Velin92 Velin92 mentioned this issue Jul 19, 2023
Closed
@Velin92
Copy link
Member

Velin92 commented Jul 19, 2023

Requires: matrix-org/synapse#15503.

@manuroe
Copy link
Member

manuroe commented Aug 31, 2023

I removed the scheduled labels. We want to it later in this epic: https://github.com/vector-im/element-internal/issues/433

@stefanceriu stefanceriu mentioned this issue Oct 10, 2023
Closed
@manuroe
Copy link
Member

manuroe commented Dec 12, 2024

Was done cross-platform in element-hq/element-meta#2549.

@manuroe manuroe closed this as completed Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-File-Upload Attachments and file uploads A-Local echo O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements X-Needs-Backend X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@kittykat @ara4n @pixlwave @manuroe @Velin92