Skip to content
This repository has been archived by the owner on Dec 7, 2023. It is now read-only.

Sync to Neo 3.0.0-CI01148 #709

Merged
merged 4 commits into from
Jan 12, 2021
Merged

Sync to Neo 3.0.0-CI01148 #709

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
da6fc35
Sync to Neo 3.0.0-CI01148
chenzhitong Jan 11, 2021
9256f6c
Remove from settings
vncoelho Jan 11, 2021
42cf648
fix
superboyiii Jan 12, 2021
cad1a81
Remove StartConsensus
shargon Jan 12, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 0 additions & 18 deletions neo-cli/CLI/MainService.Consensus.cs
View file
Open in desktop

This file was deleted.

4 changes: 0 additions & 4 deletions neo-cli/CLI/MainService.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -391,10 +391,6 @@ public async void Start(string[] args)
{
Console.WriteLine($"Failed to open file \"{Settings.Default.UnlockWallet.Path}\"");
}
if (Settings.Default.UnlockWallet.StartConsensus && CurrentWallet != null)
vncoelho marked this conversation as resolved.
Show resolved Hide resolved
{
OnStartConsensusCommand();
}
Comment on lines -395 to -397
Copy link
Member

@superboyiii superboyiii Jan 12, 2021 •
edited
Loading

Choose a reason for hiding this comment

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

We should reserve this, it will implement Plugin.GetService<IConsensusProvider>()?.Start(CurrentWallet); from dBFT plugin, MainService.Consensus.cs should be reserved as well. Wait for GetService to be modified from protected.

Copy link
Member

Choose a reason for hiding this comment

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

We should start the consensus with a new command, registered in the plugin

Copy link
Member

Choose a reason for hiding this comment

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

Agree with @shargon

}
}

Expand Down
1 change: 0 additions & 1 deletion neo-cli/config.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@
"UnlockWallet": {
"Path": "",
"Password": "",
"StartConsensus": false,
"IsActive": false
},
"PluginURL": "https://github.com/neo-project/neo-modules/releases/download/v{1}/{0}.zip"
Expand Down
1 change: 0 additions & 1 deletion neo-cli/config.mainnet.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@
"UnlockWallet": {
"Path": "",
"Password": "",
"StartConsensus": false,
"IsActive": false
},
"PluginURL": "https://github.com/neo-project/neo-modules/releases/download/v{1}/{0}.zip"
Expand Down
1 change: 0 additions & 1 deletion neo-cli/config.testnet.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@
"UnlockWallet": {
"Path": "",
"Password": "",
"StartConsensus": false,
"IsActive": false
},
"PluginURL": "https://github.com/neo-project/neo-modules/releases/download/v{1}/{0}.zip"
Expand Down