Skip to content

Update to Matrix 1.16 - #889

Draft
KitsuneRal wants to merge 8 commits into
devfrom
kitsune/update-api
Draft

Update to Matrix 1.16#889
KitsuneRal wants to merge 8 commits into
devfrom
kitsune/update-api

Conversation

@KitsuneRal

Copy link
Copy Markdown
Member

+Reformatting according to updated .clang-format, making the PR look bigger than it really is. The key things are in 91c672c.

@KitsuneRal KitsuneRal added the enhancement A feature or change request for the library label Jun 10, 2025
@sonarqubecloud

Copy link
Copy Markdown

@KitsuneRal
KitsuneRal marked this pull request as draft June 11, 2025 06:18
Note: aside from changes in gtad.yaml, matrix-spec was also changed to
suppress unneeded QJsonObjects by new jobs in report_content.h
@TobiasFella

Copy link
Copy Markdown
Member

This breaks compilation of neochat in a few places. Ifdefing those is easy enough, but it would help a lot to have a preprocessor definition that contains the version of the matrix api

@KitsuneRal

KitsuneRal commented Jul 1, 2025

Copy link
Copy Markdown
Member Author

If it's because of RoomKeysUpdateResponse then I'm working to unroll it back (unfortunately, I've hit GTAD limitations, got to write more code there first... halfway through). That's actually why this PR is still a draft.

@TobiasFella

Copy link
Copy Markdown
Member

It's not RoomKeysUpdateResponse. One is ReportContentJob being gone, the others are two cases of QStrings now being QUrls

@KitsuneRal

Copy link
Copy Markdown
Member Author

Ah, doh, ReportContentJob basically means using the older API definition, yeah.

@KitsuneRal KitsuneRal changed the title Update to Matrix 1.14 Update to Matrix 1.16 Aug 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement A feature or change request for the library

Projects

Status: In work

Development

Successfully merging this pull request may close these issues.

2 participants