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

Warn user of duplicate official name #1647

Closed
vinkabuki opened this issue Jul 19, 2023 · 3 comments
Closed

Warn user of duplicate official name #1647

vinkabuki opened this issue Jul 19, 2023 · 3 comments

Comments

@vinkabuki
Copy link
Contributor

No description provided.

@vinkabuki vinkabuki added this to Quiet Jul 19, 2023
@vinkabuki vinkabuki converted this from a draft issue Jul 19, 2023
@kingalg
Copy link
Collaborator

kingalg commented Aug 1, 2023

Link to figma designs: https://www.figma.com/file/TV9pF84Ob8pLYRLu83gNol/Joining-when-owner-is-offline?type=design&node-id=7-2510&mode=design

Screenshot of designs from figma:

Screenshot 2023-08-01 at 15 43 31

Details as per discussion in #1340:

  • When there are two or more official users with the same name, we should show a very aggressive full screen warning such as "Warning: impersonation attack. The community owner has given two people the same name. This should never happen, and it means the owner is malicious or compromised."
  • "Warning: impersonation attack" should appear "If the owner hands out duplicate names (which should never happen) it should be very visible. On every restart, for example." @holmesworcester as we are about to work on this issue could you confirm:
  1. if this should happen on every restart?
  2. are there any other situations in which it should appear?

@holmesworcester
Copy link
Contributor

It should show as soon as we notice it.

On desktop it should show on every restart, but on mobile 'restart' means something different.

I think it makes sense to redesign this to be an alert banner that displays all the time rather than a full screen thing. Or it could be a full screen warning followed by an alert banner that displays all the time. Let's knock this back to 'needs design' and move it out of the sprint.

@holmesworcester holmesworcester moved this from Next Sprint to Backlog - Mobile in Quiet Aug 1, 2023
@holmesworcester
Copy link
Contributor

duplicate of #119. closing.

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

No branches or pull requests

3 participants