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

EI: Use 3PID capability to show / hide email UI in settings for MSC3824 OIDC-aware #7645

Closed
hughns opened this issue Aug 18, 2023 · 0 comments · Fixed by #7655
Closed

EI: Use 3PID capability to show / hide email UI in settings for MSC3824 OIDC-aware #7645

hughns opened this issue Aug 18, 2023 · 0 comments · Fixed by #7655
Assignees
Labels
A-Authentication T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements

Comments

@hughns
Copy link
Member

hughns commented Aug 18, 2023

This is a sub item from #7644

When the homeserver is configured in MSC3861 delegated OIDC mode (e.g. to MAS) the user needs to add and remove 3pids via the UI provided by MAS.

As such, Element iOS should check the capability that instructs clients whether or not they can manage 3PIDs: https://spec.matrix.org/v1.7/client-server-api/#m3pid_changes-capability

If the capability is not set to true then the client should hide the UI for managing emails.

n.b. the Identity Server functionality should remain the same as today.

@hughns hughns changed the title Use 3PID capability to show / hide email UI in settings for MSC3824 OIDC-aware EI: Use 3PID capability to show / hide email UI in settings for MSC3824 OIDC-aware Aug 18, 2023
@pixlwave pixlwave added T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements A-Authentication labels Aug 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Authentication T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements
Projects
Status: Closed
Development

Successfully merging a pull request may close this issue.

3 participants