Skip to content

MyDramaList: migrate to kotlinx serialization#2870

Open
Luna712 wants to merge 2 commits into
recloudstream:masterfrom
Luna712:migrate1
Open

MyDramaList: migrate to kotlinx serialization#2870
Luna712 wants to merge 2 commits into
recloudstream:masterfrom
Luna712:migrate1

Conversation

@Luna712
Copy link
Copy Markdown
Contributor

@Luna712 Luna712 commented Jun 3, 2026

Note: MyDramaList has other preexisting issues that need fixed to make it fully work.
Note 2: We have to change things like Any to JsonElement which means our original plan to compare exact outputs likely won't work.

@Luna712 Luna712 changed the title Migrate MyDramaList metaprovider to Kotlinx Serlization MyDramaList: migrate to kotlinx serialization Jun 3, 2026
Copy link
Copy Markdown
Collaborator

@fire-light42 fire-light42 left a comment

Choose a reason for hiding this comment

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

I am unsure if this might cause problems with extensions targeting the "new" version breaking things for the stable. Do you think this should be merged before or after the next stable?

@Luna712
Copy link
Copy Markdown
Contributor Author

Luna712 commented Jun 4, 2026

I am unsure if this might cause problems with extensions targeting the "new" version breaking things for the stable. Do you think this should be merged before or after the next stable?

@fire-light42

This one in particular doesn't matter and can be merged whenever as MDL doesn't work right now. Hopefully we can fix it soon though. The others I think should be merged just before next stable especially #2874 so that extensions can use those as serializable classes.

#2800 should also probably be merged just before next stable if you're happy with it after testing etc... as it does the only other metaprovider not in my very recent PRs to kotlinx serialization also. Then only extractors will remain in library anyway.

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.

2 participants