Skip to content
This repository has been archived by the owner on Oct 16, 2020. It is now read-only.

Created an Annotation for Registering Apps #131

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

Created an Annotation for Registering Apps #131

wants to merge 4 commits into from

Conversation

AlexCouch
Copy link

I went ahead and created that annotation you were talking about. It was pretty simple tbh. I didn't have to changed anything in the original code, just had to provide a way for the app class to be pushed from the annotation backend to the actual registration system: ApplicationManager#registerApplication. Created a simple wrapper interface that gets implemented when the class gets hooked through the annotation. The annotation's information gets passed into said interface, IAppContainer, as well as the class instance of the app class gets stored in the interface instance as well to be passed into the original registration system.

@ghost
Copy link

ghost commented May 23, 2018

Great idea! I hope this gets accepted

@AlexCouch
Copy link
Author

Well, cray has already made his own version while I was making this. But I'm currently finalizing a plugin system for adding support for different uses of an annotation @CDMRegister for different elements of the mod/addons. Right now, applications and tasks are supported.

…support for autoregistering things for device mod
@ghost
Copy link

ghost commented May 23, 2018

Ok

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant