Resolves 1 from ENG-9345.
We shouldn't add loop property when updating photo multimedia message. Otherwise, the DM Ops validator would reject such message.
Details
Details
Quickly-sent a DM and made sure the payload doesn't contain the loop property the DM op is processed successfully on recipient side.
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
Comment Actions
Is there a risk here that the loop property isn't being populated correctly where it should be? I believe this is meant to be used for video, where we want the video to autoplay, and then loop back to the start when it finishes and continue playing. Not sure it works correctly, but ideally we can avoid breaking it more
Comment Actions
Yes it's used only for videos. I made sure that here in web input-state-container it's only used for upload calls and not for displaying videos. Since we're not uploading videos from web, it won't break anything.