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.

Sign up for GitHub

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

Jump to bottom

Add a new tab to user page with room memberships #73

Merged
merged 2 commits into from
Feb 11, 2021
Merged

Add a new tab to user page with room memberships #73

merged 2 commits into from
Feb 11, 2021

Conversation

dklimpel
Copy link
Contributor

The admin API for user's room memberships is new in synapse v1.21.0.

@awesome-manuel
Copy link
Member

awesome-manuel commented Nov 16, 2020 •
edited
Loading

I get HTTP 404 with {"errcode":"M_NOT_FOUND","error":"User not found"} for all users using synapse 1.21.2.

@dklimpel
Copy link
Contributor Author

I did a new test. I am get HTTP 200 and in console:

dataProvider.js:205 getManyReference joined_rooms
dataProvider.js:7 httpClient https://server/_synapse/admin/v1/users/@testuser:server/joined_rooms

@awesome-manuel
Copy link
Member

Which synapse version did you use?

@dklimpel
Copy link
Contributor Author

Synapse v1.22.1.
The API is realeased in v1.21.0: https://github.com/matrix-org/synapse/blob/develop/CHANGES.md#features-3
matrix-org/synapse#8306

@awesome-manuel
Copy link
Member

Could it be a problem of users that are in no room (yet)?

@dklimpel
Copy link
Contributor Author

Yes, you're right.

@dklimpel
Copy link
Contributor Author

It is fixed in Synapse v1.23.0.
matrix-org/synapse#8643

@dklimpel dklimpel mentioned this pull request Dec 23, 2020
Open
@dklimpel
Add a new tab to user page with room memberships
c029ddb 
The admin API for user's room memberships is new in synapse v1.21.0.
@awesome-michael awesome-michael merged commit 706114a into Awesome-Technologies:master Feb 11, 2021
@dklimpel dklimpel deleted the room_membership branch February 11, 2021 20:14
@dklimpel
Copy link
Contributor Author

Related #100

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dklimpel @awesome-manuel @awesome-michael