Skip to content

MSC4260: Reporting users (Client-Server API)#2093

Merged
turt2live merged 7 commits intomatrix-org:mainfrom
Johennes:johannes/msc4260
Mar 21, 2025
Merged

MSC4260: Reporting users (Client-Server API)#2093
turt2live merged 7 commits intomatrix-org:mainfrom
Johennes:johannes/msc4260

Conversation

@Johennes
Copy link
Copy Markdown
Contributor

@Johennes Johennes commented Mar 13, 2025

Spec for matrix-org/matrix-spec-proposals#4260

Pull Request Checklist

Preview: https://pr2093--matrix-spec-previews.netlify.app

Signed-off-by: Johannes Marbach <n0-0ne+github@mailbox.org>
Comment thread data/api/client-server/report_content.yaml Outdated
Comment thread content/client-server-api/modules/report_content.md Outdated
Comment thread content/client-server-api/modules/report_content.md Outdated
@Johennes Johennes marked this pull request as ready for review March 13, 2025 10:07
@Johennes Johennes requested a review from a team as a code owner March 13, 2025 10:07
Comment thread data/api/client-server/report_content.yaml
Co-authored-by: Kévin Commaille <76261501+zecakeh@users.noreply.github.com>
Copy link
Copy Markdown
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

otherwise lgtm - thanks for writing it up!

Comment thread content/client-server-api/modules/report_content.md Outdated
Comment thread content/client-server-api/modules/report_content.md Outdated
Comment thread data/api/client-server/report_content.yaml Outdated
@Johennes Johennes requested a review from turt2live March 21, 2025 09:16
@Johennes
Copy link
Copy Markdown
Contributor Author

I have opened matrix-org/matrix-spec-proposals#4277 for future alignment of the three endpoints.

Copy link
Copy Markdown
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, and sorry for the thrashing on required - future MSCs will hopefully document these differences better.

@turt2live turt2live merged commit c758135 into matrix-org:main Mar 21, 2025
12 checks passed
@Johennes Johennes mentioned this pull request Jul 2, 2025
30 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants