Skip to content

Conversation

KitsuneRal
Copy link
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
Copy link

@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
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
Copy link
Member Author

KitsuneRal commented Jul 1, 2025

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
Member

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

@KitsuneRal
Copy link
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