Points 3a and 3b from ENG-6459.
Details
- Reviewers
ashoat - Commits
- rCOMM0b058dcfae34: [lib] Stop shimming Blob-hosted multimedia messages
Replaced next code version constant with a past version and verified that robotext no longer appears and blob-hosted media messages are displayed.
Diff Detail
- Repository
- rCOMM Comm
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
lib/shared/messages/multimedia-message-spec.js | ||
---|---|---|
256 ↗ | (On Diff #45655) | We've never been shimming these for web. This code was written before we started versioning web |
First, give this comment a read.
Basically my thinking is that if we want to have clients immediately start sending blob-hosted multimedia in thin threads, we should also stop shimming blob-hosted multimedia on the keyserver for recent codeVersions.
Rather than only unshimming for the most recent codeVersions, we should instead unshim for any codeVersion that can support blob-hosted multimedia.
lib/shared/messages/multimedia-message-spec.js | ||
---|---|---|
256 | Please replace NEXT_CODE_VERSION here with the earliest codeVersion that supports blob-hosted multimedia. (Would be helpful to explain your reasoning) |