The Wayback Machine - https://web.archive.org/web/20200714060936/https://github.com/vector-im/riot-web/issues/11213
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

Switch to verifying users rather than devices #11213

Closed
dbkr opened this issue Oct 24, 2019 · 2 comments
Closed

Switch to verifying users rather than devices #11213

dbkr opened this issue Oct 24, 2019 · 2 comments

Comments

@dbkr
Copy link
Member

@dbkr dbkr commented Oct 24, 2019 •

This probably ought to be coupled tightly to #11212 since between doing that and this, we'll be in a weird quasi-state where we've set up cross-signing but new verifications will still be verifying devices.

In the MemberInfoPanel, if the cross-signing labs flag is enabled, hide verify buttons on the individual devices (moved to #11668) and instead show just one for the user. This button does a cross-signing verification. Use existing verification UI for now, placeholder UI for the verify button at this stage.

@jryans
Copy link
Member

@jryans jryans commented Dec 14, 2019

matrix-org/matrix-react-sdk#3728 is enough to verify users, but it does not currently hide the devices as well. From chat in the room, we may want a separate issue to track hiding the device list in case it's more complex than just hiding?

@jryans
Copy link
Member

@jryans jryans commented Dec 14, 2019

Or does "hide verify buttons on the individual devices" mean the device list would stay, but just the verify buttons per device removed...? Either way, let's track with a separate issue: #11668

Workflow automation moved this from In Review to In Test Dec 16, 2019
@jryans jryans added the phase:1 label Feb 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Workflow
In Test
Linked pull requests

Successfully merging a pull request may close this issue.

3 participants
You can’t perform that action at this time.