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

Cannot use "Chat" button on another user profile until accepting contact request #14541

Closed
churik opened this issue Dec 14, 2022 · 3 comments · Fixed by #14726
Closed

Cannot use "Chat" button on another user profile until accepting contact request #14541

churik opened this issue Dec 14, 2022 · 3 comments · Fixed by #14726
Assignees
Milestone

Comments

@churik
Copy link
Member

churik commented Dec 14, 2022

Bug Report

Problem

As currently the design for other user profiles (stranger) in WIP state current user profile will be used instead.
After #14492 is merged, there is no way to chat user who did not accept your contact request.

Expected behavior

Cannot send message to the user until sending CR
Suggested for now: disable "Chat" button until sending and approving CR

Also in the scope of the same issue "Mutual contact request" (Profile > Settings > Advanced) should be enabled by default

Actual behavior

Can send a message and it won't be delivered
Monosnap 2022-12-14 17-39-42

Reproduction

  1. Open Status
  2. Join any community or pub channel
  3. Tap on another user profile (not contact)
  4. Tap on "Chat"

Additional Information

  • Status version: nightly
  • Operating System: Android, iOS
@churik churik added this to the 1.21.0-rc.1 milestone Dec 14, 2022
@ibrkhalil
Copy link
Contributor

@cammellos Should we disable accessing a 1-on-1 with the user until they're mutually have accepted a CR ?

@cammellos
Copy link
Contributor

@ibrkhalil yes

@ibrkhalil ibrkhalil self-assigned this Dec 19, 2022
@ibrkhalil
Copy link
Contributor

Okay, Will create a PR to do it

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 a pull request may close this issue.

3 participants