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

Major rewrite #40

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

Major rewrite #40

wants to merge 16 commits into from

Conversation

Plastikmensch
Copy link
Owner

@Plastikmensch Plastikmensch commented Oct 10, 2022

Closes #35, #32
Fixes #33, #36, #37, #39

Complete rewrite of this plugin, more versatile code and bug fixes.

Instead of having to define each entity individually, get entities directly from the EntityType enum, which includes every entity.
Infer every relevant entity information from the entities interface, instead of doing it manually, with the option to change values as needed.
Run entity updates on a timer instead of repeatedly creating new tasks.
Renamed variables to more clearly indicate their use.
Changed internal entity IDs.
Created modules for better readability of source.

Config needs to be deleted if an earlier version was used

@Plastikmensch Plastikmensch marked this pull request as ready for review October 19, 2022 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Code refactors
Projects
None yet
1 participant