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

m.get_login_token capability is not documented #1547

Open
hughns opened this issue May 26, 2023 · 1 comment · May be fixed by #1908
Open

m.get_login_token capability is not documented #1547

hughns opened this issue May 26, 2023 · 1 comment · May be fixed by #1908
Labels
spec-bug Something which is in the spec, but is wrong

Comments

@hughns
Copy link
Member

hughns commented May 26, 2023

Link to problem area:
https://spec.matrix.org/v1.7/client-server-api/#get_matrixclientv3capabilities

Issue
https://spec.matrix.org/v1.7/client-server-api/#post_matrixclientv1loginget_token makes reference to a m.get_login_token capability. However, the capability section 5 does not describe the capability: https://spec.matrix.org/v1.7/client-server-api/#get_matrixclientv3capabilities

image

Expected behaviour

The m.get_login_capability should be described in section 5.

@richvdh
Copy link
Member

richvdh commented May 26, 2023

for archaeology: the new endpoint was added in #1530, which appear to have forgotten to document the capability in section 5.

When fixing this, don't forget to add it to the table at https://spec.matrix.org/v1.7/client-server-api/#_matrixclientv3capabilities_capabilities.

Johennes added a commit to Johennes/matrix-spec that referenced this issue Jul 12, 2024
Fixes: matrix-org#1547
Signed-off-by: Johannes Marbach <n0-0ne+github@mailbox.org>
@Johennes Johennes linked a pull request Jul 12, 2024 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec-bug Something which is in the spec, but is wrong
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants