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

feat: do not post "... verified" messages on QR scan success #4998

Merged
merged 1 commit into from
Nov 14, 2023

Conversation

link2xt
Copy link
Collaborator

@link2xt link2xt commented Nov 13, 2023

We still post "... not verified" on failure.

Closes #4994

@link2xt link2xt force-pushed the link2xt/no-contact-verified branch 2 times, most recently from 486b0b2 to 5b584e4 Compare November 14, 2023 00:01
We still post "... not verified" on failure.
@link2xt link2xt force-pushed the link2xt/no-contact-verified branch from 5b584e4 to dc2d43a Compare November 14, 2023 00:36
@link2xt link2xt merged commit c6ea4e3 into main Nov 14, 2023
36 checks passed
@link2xt link2xt deleted the link2xt/no-contact-verified branch November 14, 2023 09:59
link2xt added a commit that referenced this pull request Jan 9, 2024
The chat is not modified at least since
c6ea4e3
(PR #4998),
even the info message is not posted there.
link2xt added a commit that referenced this pull request Jan 9, 2024
It was named notify_peer_verified()
because it added info message,
but this is no longer true since
#4998
(commit c6ea4e3)
is merged.
link2xt added a commit that referenced this pull request Jan 9, 2024
The chat is not modified at least since
c6ea4e3
(PR #4998),
even the info message is not posted there.
link2xt added a commit that referenced this pull request Jan 9, 2024
It was named notify_peer_verified()
because it added info message,
but this is no longer true since
#4998
(commit c6ea4e3)
is merged.
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.

Remove "xxx verified" ("xxx introduced") system messages
2 participants