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

Extension System #251

Closed
sr229 opened this issue Oct 14, 2021 Discussed in #216 · 4 comments · Fixed by #262
Closed

Extension System #251

sr229 opened this issue Oct 14, 2021 Discussed in #216 · 4 comments · Fixed by #262
Assignees
Labels
priority:high Bugs or Features that requires immediate attention.

Comments

@sr229
Copy link
Member

sr229 commented Oct 14, 2021

Discussed in #216

Originally posted by sr229 May 9, 2021
This has been requested by the community; however, this is kinda low priority as we focus most on the core components. The way this works is the following:

  • Extensions can expose their settings in MainMenu.
  • They will be strictly be conformant to the o!f model to properly load. This is considered "bare minimum" for what people requires to make an extension.
  • They will be packaged as either a .dll or a .nupkg which the program can "extract" or "compile" into a DLL, something we can do once we have a better idea with how to dynamically load assemblies.

Anyone can propose a better design here since this is a RFC, we appreciate alternative approaches for this.

@sr229 sr229 added proposal New feature or request priority:medium Bugs or Features that may require attention sooner or later. labels Oct 14, 2021
@sr229 sr229 moved this to Todo in Encore Tracker Jan 4, 2022
@sr229 sr229 moved this from Todo to In Progress in Encore Tracker Jan 4, 2022
@sr229 sr229 moved this from In Progress to Todo in Encore Tracker Jan 4, 2022
@LeNitrous LeNitrous added priority:high Bugs or Features that requires immediate attention. and removed priority:medium Bugs or Features that may require attention sooner or later. labels Jan 6, 2022
@LeNitrous
Copy link
Member

Move to high priority as it is now considered integral to the core.

@sr229 sr229 removed the proposal New feature or request label Jan 6, 2022
@sr229
Copy link
Member Author

sr229 commented Jan 6, 2022

Escalating into a new feature.

@sr229
Copy link
Member Author

sr229 commented Jan 6, 2022

I believe @adryzz wanted to implement ACLs on this since from our specification, we do require the extensions to list their required capabilities.

For now what we require are the following capabilities implemented:

  • scene
  • tracking
  • io
  • net

@sr229
Copy link
Member Author

sr229 commented Jan 10, 2022

Here's the formalized/finalized version of our extension architecture, for more details check out GH-256.

image

@sr229 sr229 closed this as completed in #262 Mar 7, 2022
Repository owner moved this from Todo to Done in Encore Tracker Mar 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:high Bugs or Features that requires immediate attention.
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants