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.

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

Improved and updated data component wrapper #1

Open
wants to merge 80 commits into
base: master
Choose a base branch
from

Conversation

WolfyScript
Copy link
Owner

@WolfyScript WolfyScript commented Nov 6, 2024

Reworks the whole DataKey system that wraps and convert platform specific data component APIs.

This PR defines the data keys, that convert these platform types to a common API, and makes sure they are registered on each platform.

Most data keys are yet to be implemented, either because conversions need more work, or a platform does not provide an API to access the data component. e.g. there's no API for Lock components on Spigot.
A List about the current state of the component implementation can be found here in the wiki

* fix factories order of initialisation
(Sponge, Spigot not fully implemented yet)
* Locks are no longer pure strings
* Spigot has apparently no API for the new lock format (yet?)
* Need to update to new Sponge API for support. Mark as TODO for now.
* This adds a new Lock object to future-proof it
* The textures de-/serialization still needs work
* They are not implemented yet, but this way they can be tested one after another
@WolfyScript WolfyScript added the enhancement New feature or request label Nov 6, 2024
@WolfyScript WolfyScript marked this pull request as ready for review November 26, 2024 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant