Skip to content

Fix AddService#2221

Merged
erikzhang merged 2 commits intomasterfrom
remove-service
Jan 13, 2021
Merged

Fix AddService#2221
erikzhang merged 2 commits intomasterfrom
remove-service

Conversation

@erikzhang
Copy link
Member

No description provided.

Copy link
Member

@vncoelho vncoelho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wallet as a service on the plugins?
I need to study this.

@ZhangTao1596
Copy link

ZhangTao1596 commented Jan 13, 2021

We should stop service before remove or replace.
Why we need Remove and Replace? Why don't we just restart node?

@erikzhang erikzhang changed the title Allow remove/replace services Fix AddService Jan 13, 2021
@erikzhang erikzhang merged commit c5a0a72 into master Jan 13, 2021
@erikzhang erikzhang deleted the remove-service branch January 13, 2021 08:32
cloud8little pushed a commit to cloud8little/neo that referenced this pull request Jan 24, 2021
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 this pull request may close these issues.

3 participants