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

Reimplement TwitchSpawn with Architectury API #69

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

Conversation

BONNe
Copy link

@BONNe BONNe commented Dec 25, 2023

The Architectury API

This allows easy Fabric/Forge transformations and compile mods for both environments with minimal changes required.

This pull request contains all changes required to run TwitchSpawn on fabric and forge servers. The plugin's main logic is moved to the common project which is built on Architectury API.
The fabric and forge projects now contain just mod loader-specific changes, such as mixins, events and starters.

As Fabric does not have an error screen like a forge, this pull request contains the error screen for the mod config errors, which is opened when a player starts any client. The screen contains Open Config, Reload Config and Continue buttons.

There were some changes required to the localization, as Minecraft parses with %s unlike forge with {}.

Also, I fixed the sub-title that displayed the mob name, as it was not parsed, and added the Latvian locale, because I could.

BONNe added 15 commits December 21, 2023 22:44
All these changes were made to allow TwitchSpawn to run on 1.20.2.

- Update gradle version
- add settings.gradle as it is new standard for projects
- add sound event registry and augment type registry
- rework item parsing, as it is changed
- rework network manager and usage to apply 1.20 changes
- switch from performCommand to performPrefixedCommand
- fix some gui rendering to reflect 1.20 changes
- Replace TranslatableComponent to the new Component#translatable
This screen is triggered on client startup and has config errors (like in forge).
Remove unnecessary parts from the optimizing screen exceptions.
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.

1 participant