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

Invisible Crypto: EX Android: display a warning when an *unverified* user changes identity #2524

Closed
Tracked by #2544
andybalaam opened this issue Sep 9, 2024 · 4 comments
Assignees

Comments

@andybalaam
Copy link
Member

Followup to #2492. Part of #2491, itself part of Invisible crypto.

When an unverified user changes their identity, we need to make our user aware of this. In the long term, the intention is just to show a notice in the timeline (#2493); however, that is difficult to implement and we need a stop-gap.

This task is for Element X Android.

The proposal is to show a warning above the composer, in much the same way as we would for verified users (#2491), but without locking the composer.

Figma designs:

Image

@mxandreas
Copy link

For visibility: #2525 (comment)

@manuroe
Copy link
Member

manuroe commented Oct 8, 2024

@bmarty can we close this issue thanks to element-hq/element-x-android#3621?

@bmarty
Copy link
Member

bmarty commented Oct 10, 2024

Yes, indeed, this is done in element-hq/element-x-android#3621

This task is for Element X Android.

@andybalaam for next time, can you create the task to the EXA project https://github.com/element-hq/element-x-android please? The created issues can then be added to the story, #2544 in this case. The story was in the correct project element-meta. Thanks!

This also apply to #2525 which should have been created on the EXI project https://github.com/element-hq/element-x-ios.

It will reduce confusion since I see that comments have been added to the iOS task and not to the story, for instance this one: #2525 (comment). CC @mxandreas.

Thanks all!

@bmarty bmarty closed this as completed Oct 10, 2024
@andybalaam
Copy link
Member Author

@andybalaam for next time, can you create the task to the EXA project

Thanks for the heads up. We actually recently changed our processes so future tasks should be in the correct project. Sorry for the time-waste this time.

Thanks!

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

4 participants