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

Element doesn't let users bind their emails with identity servers if the m.3pid_changes capability is off #28355

Closed
sandhose opened this issue Oct 31, 2024 · 0 comments · Fixed by #28375
Assignees
Labels
O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@sandhose
Copy link
Member

Steps to reproduce

  1. Use a server which advertises the m.3pid_changes capability to false, like beta.matrix.org
  2. Somehow have an email address associated with the account
  3. Try to share that email with an identity server

Outcome

What did you expect?

Be able to share the email

What happened instead?

The button is greyed out
image


The faulty logic is probably here:

const capabilities = await client.getCapabilities();
setCanMake3pidChanges(
!capabilities["m.3pid_changes"] || capabilities["m.3pid_changes"].enabled === true,
);

Operating system

macOS

Application version

Element Nightly version: 2024103101 Crypto version: Rust SDK 0.7.2 (517d99b), Vodozemac 0.7.0

How did you install the app?

No response

Homeserver

Synapse 1.117.0 with MAS 0.12.0

Will you send logs?

No

@dosubot dosubot bot added O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist labels Oct 31, 2024
@t3chguy t3chguy self-assigned this Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants