Page MenuHomePhabricator

[web] Avoid keyserver call when deleting media for DMs
ClosedPublic

Authored by bartek on Sep 11 2024, 1:27 AM.
Tags
None
Referenced Files
F4080649: D13282.id44024.diff
Sat, Feb 15, 9:30 PM
F4080648: D13282.id44162.diff
Sat, Feb 15, 9:30 PM
F4080602: D13282.id.diff
Sat, Feb 15, 9:29 PM
Unknown Object (File)
Sat, Feb 1, 12:22 AM
Unknown Object (File)
Jan 15 2025, 6:16 AM
Unknown Object (File)
Jan 8 2025, 9:17 AM
Unknown Object (File)
Jan 6 2025, 4:07 PM
Unknown Object (File)
Dec 27 2024, 1:43 AM
Subscribers

Details

Summary

On web, we pre-upload media as soon as it is selected, before actually sending the message. When user decides not to send the image (clicks the cross button in the corner),
the uploaded media gets deleted. This results in keyserver call: this.props.deleteUpload(). For DMs, we want to avoid such call.

Needed to update some code to pass the whole threadInfo of active chat, instead of only threadID.

Depends on D13281

Test Plan
  • Verified that behavior for keyserver media hasn't been changed and the endpoint is still called
  • TBD for DMs

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage