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

Support setting per-plugin configuration options in the config file #54

Closed
markgoddard opened this issue Nov 29, 2024 · 0 comments · Fixed by #55
Closed

Support setting per-plugin configuration options in the config file #54

markgoddard opened this issue Nov 29, 2024 · 0 comments · Fixed by #55
Assignees
Milestone

Comments

@markgoddard
Copy link
Contributor

For some plugins it would be useful to be able to store configuration options in cofide.yaml. These may be arbitrarily structured. Several plugins may exist, and should each be able to add their own config.

@markgoddard markgoddard self-assigned this Nov 29, 2024
@markgoddard markgoddard added this to the release-0.7.0 milestone Nov 29, 2024
markgoddard added a commit that referenced this issue Nov 29, 2024
For some plugins it would be useful to be able to store configuration
options in cofide.yaml. This change updates the plugin manager to
support setting and retrieving per-plugin configuration keyed by the
plugin name. The config can be set during Init or later using
GetPluginConfig.

A Struct protobuf message is used to allow for arbitrarily structured
plugin config. Plugins can use the Struct directly or convert it to/from
a more convenient native Go type.

Fixes: #54
markgoddard added a commit that referenced this issue Nov 29, 2024
For some plugins it would be useful to be able to store configuration
options in cofide.yaml. This change updates the plugin manager to
support setting and retrieving per-plugin configuration keyed by the
plugin name. The config can be set during Init or later using
GetPluginConfig.

A Struct protobuf message is used to allow for arbitrarily structured
plugin config. Plugins can use the Struct directly or convert it to/from
a more convenient native Go type.

Fixes: #54

Depends-On: cofide/cofide-api-sdk#8
markgoddard added a commit that referenced this issue Dec 2, 2024
Support per-plugin configuration
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 a pull request may close this issue.

1 participant