MyDramaList: migrate to kotlinx serialization#2870
Conversation
fire-light42
left a comment
There was a problem hiding this comment.
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?
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. |
Note: MyDramaList has other preexisting issues that need fixed to make it fully work.
Note 2: We have to change things like
AnytoJsonElementwhich means our original plan to compare exact outputs likely won't work.