A cursory glance shows that createAsyncMigrate handles calling handleReduxMigrationFailure when an exception is thrown. I wish both @tomek and @bartek had looked at the code a little closer here.

It seems like we're discarding the processed media, since we're returning uploadURI to the caller (non-processed media). I hope I'm missing something here, as both @angelika and @bartek did not flag this...

Fix the logic

Accepting pending resolution of CI

Reduce number of queries

Rebase

Return deleted message when fetching related messages

Fix deleting messages in a sidebar

Update a comment

Handle migration failure

Rebase

Rebase

Remove unnecessary condition

Optimize the code a bit

Looks a little weird, but it's okay

I suppose the italics don't change the height

Simplify

In D14547#403433, @ashoat wrote:

Can you share a screenshot of what a mouseover looks like in this case? Wondering if the timestamp still appears, and where it appears

Rebase

Delete outdated comment

Don't use markdown when measuring deleted messages

Avoid using markdown. Update the design.

In D14544#403424, @ashoat wrote:

The white background feels like too strong of a contrast... I feel like a deleted message should be less visible than a non-deleted message. Could we try something closer to what Rahul had in his designs?

Simplify. Update the style.

Simplify

Rebase

In D14542#403439, @ashoat wrote:

I want to review the icon before landing, but this generally looks good

Use the util and replace the web icon

Await the promise to make this symmetric with the thick thread's code

Simplify the code

In D14536#403268, @ashoat wrote:

Do we need to do anything for it to be included in the Android build, eg. update some CMakeLists.txt file?

Rebase

Rebase

Rebase

Rebase

Close the tooltip after deleting a message