Skip to content

Conversation

cafalchio
Copy link

This commit is a fix to issue #150 and also adding icons to show pending and failed messages.

image

@HocusLocusTee
Copy link
Collaborator

HocusLocusTee commented Jul 22, 2025

I think this helps towards Issue . But we can still send / spam messages (leading to failures) and its not really fix.

Further more to this, if you send a message before the fee estimation is finished. You can get toast errors.

@HocusLocusTee
Copy link
Collaborator

HocusLocusTee commented Jul 22, 2025

Also, in some sates we get messages that have none of the status symbols:
image

See my comment below, it might address this.

@HocusLocusTee
Copy link
Collaborator

Another discussion point:
If I try to send a tx and fee estimation is not complete. It just sends the tx and fails it.
Leaving the message text still in the input.

I think this is going to be a v0.4.0 addition.
I don't want to rush this through just because we have a deadline with v0.3.0

image

@cafalchio
Copy link
Author

Do you think we need to wait for estimated fee before letting the user to click send?

@HocusLocusTee
Copy link
Collaborator

Do you think we need to wait for estimated fee before letting the user to click send?
Thats how it currently is on staging.

I think there is a better solution somewhere.
Even if we don't wait.. we shouldn't do the following: fail, not retry sending and leave a failed message bubble.

@cafalchio
Copy link
Author

100% agree.

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

Successfully merging this pull request may close these issues.

2 participants