Skip to content
This repository has been archived by the owner on Dec 8, 2019. It is now read-only.

Message delivered indication #22

Open
deavmi opened this issue Jul 22, 2015 · 7 comments
Open

Message delivered indication #22

deavmi opened this issue Jul 22, 2015 · 7 comments

Comments

@deavmi
Copy link

deavmi commented Jul 22, 2015

Maybe it would be nice to have message deliverance notifications like in whatsapp. Get the protocol to send a message to the sender telling him that the message has been sent.

@deavmi
Copy link
Author

deavmi commented Jul 23, 2015

Was about to say. This could help determine weather the phoen should stirebthe messages locally until the node comes online again.

@Nutomic
Copy link
Owner

Nutomic commented Jul 23, 2015

This is a bit more complicated because messages can always be lost. So it could be that a message arrived, but the confirmation gets lost. And you have no way to know about that. And retrying forever also doesn't seem like a good idea.

@deavmi
Copy link
Author

deavmi commented Jul 24, 2015

But surely you could say the same for the message getting lost.

Sent from my iPod

On Jul 24, 2015, at 12:57 AM, Felix Ableitner [email protected] wrote:

This is a bit more complicated because messages can always be lost. So it could be that a message arrived, but the confirmation gets lost. And you have no way to know about that. And retrying forever also doesn't seem like a good idea.


Reply to this email directly or view it on GitHub.

@Nutomic
Copy link
Owner

Nutomic commented Jul 24, 2015

That's right, which makes this even more complicated. We might want to wait until #26 is fixed.

@deavmi
Copy link
Author

deavmi commented Jul 24, 2015

Okay. Also is a dynamic routing protocol planned?

Sent from my iPod

On Jul 24, 2015, at 4:13 PM, Felix Ableitner [email protected] wrote:

That's right, which makes this even more complicated. We might want to wait until #26 is fixed.


Reply to this email directly or view it on GitHub.

@Nutomic Nutomic changed the title Message delivered verification Message delivered indication Oct 1, 2015
@Nutomic Nutomic added gui and removed protocol labels Jul 17, 2016
@Nutomic
Copy link
Owner

Nutomic commented Jul 17, 2016

This is now implemented in the protocol, but not displayed in the app yet.

@deavmi
Copy link
Author

deavmi commented Jul 17, 2016

Awesome work @Nutomic.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants