-
Notifications
You must be signed in to change notification settings - Fork 39
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
feat(plugins/Subscription): Manage GOOSE service connections #569
Conversation
Updating an element is done as follows:
I want this to be done with 1 newActionEvent instead of 2, but I can't get it to work. It's working as it is now, but it's not a classy solution. You're free to fix it :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
… GOOSE should be deleted
Also adding some tests for the subscription plugin, which were not included in the subscriber-plugin branch.