ChatView: implement white check mark for messages

Now that SwarmMessage got a status (sending/sent)

Note: only the last sent got the mark

https://xd.adobe.com/view/2c67dd11-99d4-4cf8-b38b-8b5797dd91c1-7934/screen/ea7b1d3f-564b-4564-98e4-e30535049fb0/

added good-first-issue UI/UX labels

added SprintIn progress label

assigned to @pnicolas

Daemon API:

Entry points are:

swarmMessageReceived

swarmMessageUpdated

AccountMessageStatusChanged

swarmLoaded

Implemented and merged on Qt: https://review.jami.net/c/jami-client-qt/+/27681

Not sure but looks like implemented on iOs too: https://review.jami.net/c/jami-client-ios/+/27741

@sblin

Now that SwarmMessage got a status (sending/sent)

The circle means that the message is currently being sent?

Note: only the last sent got the mark

Why does only the last message get a mark?

How can I find out whether one of the previous messages has not yet been sent if the status of whether a message has been sent is only displayed for the last message?

circle = sending, white check mark = sent, read status = displayed

Why does only the last message get a mark?

Cause the history is a tree (a git repository behind), so if you send A,B,C if contact got C they will have A & B (because the tree is validated and can't be partial)

Cause the history is a tree (a git repository behind), so if you send A,B,C if contact got C they will have A & B (because the tree is validated and can't be partial)

Does this mean that if there is a problem with the sending of A or B, C will not be transmitted either? So first A, then B, then C. And only if the previous message could be sent the last message is also sent?

p.s. The desktop version has the circle for sending, but no marker on the last message, right?

Desktop got the same modification. It should be available since today.

A git fetch will try to get the tree from the last commit. If the fetch fails there is no new message. Else A,B,C will be announced

If you see A,C and not B, it's a UI bug (refreshing the view should be enough) because backend will have ABC

Patch for desktop is there: https://review.jami.net/c/jami-client-qt/+/27681

MessageStates enum (daemon)

changed the description

mentioned in commit a02538f2

mentioned in commit acf2635f

mentioned in commit 02333eb8

removed SprintIn progress label

closed