Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feature/noid/migrate to kotlinx #3699

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Feature/noid/migrate to kotlinx #3699

wants to merge 3 commits into from

Commits on Mar 7, 2024

  1. convert RestModule to kt 

    Signed-off-by: Marcel Hibbe <[email protected]>
    @mahibi
    mahibi committed Mar 7, 2024
    Configuration menu
    Copy the full SHA
    a576e5f View commit details
    Browse the repository at this point in the history

  2. add KotlinxJodaTimeConverter 

    Signed-off-by: Marcel Hibbe <[email protected]>
    @mahibi
    mahibi committed Mar 7, 2024
    Configuration menu
    Copy the full SHA
    3d7a72f View commit details
    Browse the repository at this point in the history

  3. WIP. migrate from logansquare to kotlinx 

    This commit is breaking the build for now!

There are added some TODOS that need to be resolved if migration is continued!!!!

This migration was initiated because of federation: it seemed to be necessary that RoomLastMessage(json) must be able to be of type ChatMessage and RoomProxyMessage, which is problematic with logansquare. Because of this (and because logansquare is outdated anyway), migration to kotlinx was started.
However it turned out that parsing to RoomProxyMessage is not necessary (for now?!).

So for now, work is paused (and might not be continued as there might be a migration to OpenAPI instead in the future).

Signed-off-by: Marcel Hibbe <[email protected]>
    @mahibi
    mahibi committed Mar 7, 2024
    Configuration menu
    Copy the full SHA
    50e5c17 View commit details
    Browse the repository at this point in the history