Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Allow to save a draft if the verification is broken #4542

Merged
merged 1 commit into from
Jul 16, 2023

Conversation

Hocuri
Copy link
Collaborator

@Hocuri Hocuri commented Jul 12, 2023

If the verification is broken, can_send() is false.

But if the user was typing a message right when a verification-breaking message came in, the UI still needs to be able to save it as a draft.

Steps to reproduce the bug:

  • Set a draft
  • Your chat partner breaks verification
  • Go back to the chats list
  • Go to the chat again
  • Accept the breakage
  • Expected: The draft is still there
  • Bug behavior: The draft is gone

If the verification is broken, `can_send()` is false.

But if the user was typing a message right when a verification-breaking
message came in, the UI still needs to be able to save it as a draft.
@Hocuri Hocuri requested a review from link2xt July 12, 2023 14:34
@Hocuri Hocuri mentioned this pull request Jul 12, 2023
13 tasks
@Hocuri
Copy link
Collaborator Author

Hocuri commented Jul 15, 2023

@link2xt ping :) Very quick to review.

@Hocuri Hocuri merged commit d762753 into master Jul 16, 2023
@Hocuri Hocuri deleted the hoc/allow-saving-draft-when-cant-send branch July 16, 2023 10:04
@link2xt link2xt mentioned this pull request Oct 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants