IntellectualSites / ClipboardConnect

Handles clipboards for players across the network
GNU General Public License v3.0
3 stars 2 forks source link

Convert ClipboardMessage class from Kotlin to Java #13

Closed TheMeinerLP closed 10 months ago

TheMeinerLP commented 10 months ago

Proposed changes

ClipboardMessage class has been migrated from Kotlin to Java corresponding changes were made to the SyncService class. The code uses TypedJsonJacksonCodec instead of JsonJacksonCodec and direct getter methods were used for extracting userId and fromServer data from the message. The uncaught exceptions also now handled properly.

Types of changes

What types of changes does your code introduce to this project? Put an x in the boxes that apply

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

Further comments

If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...