From 4ceca1c3b459a511a040a483022150d8a6eddc62 Mon Sep 17 00:00:00 2001
From: SondreB <sondre@outlook.com>
Date: Tue, 9 Mar 2021 10:32:21 +0100
Subject: [PATCH] Minor improvement to the NodeBuilder for test node.

- Ensures no DRY.
- X42 should not use the WatchOnlyWallet, not being maintained or used.
- Update the changelog.
---
 CHANGELOG.md                           | 14 ++++++++++++++
 src/Node/Blockcore.Node/NodeBuilder.cs | 12 +++++-------
 2 files changed, 19 insertions(+), 7 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index e8a5f1348..0157eddc3 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,3 +1,17 @@
+##  (2021-03-09)
+
+* Add IMPLX MainNet SeedNodes and CheckpointInfo (#283) ([8595042](https://github.com/block-core/blockcore/commit/8595042)), closes [#283](https://github.com/block-core/blockcore/issues/283)
+* Add upload of artifact to the build (#292) ([6710591](https://github.com/block-core/blockcore/commit/6710591)), closes [#292](https://github.com/block-core/blockcore/issues/292) [#291](https://github.com/block-core/blockcore/issues/291)
+* Added support for xRhodium (XRC) (#286) ([6430223](https://github.com/block-core/blockcore/commit/6430223)), closes [#286](https://github.com/block-core/blockcore/issues/286)
+* Adding HomeCoin Network  (#289) ([129c67d](https://github.com/block-core/blockcore/commit/129c67d)), closes [#289](https://github.com/block-core/blockcore/issues/289)
+* bump version ([b8d16e9](https://github.com/block-core/blockcore/commit/b8d16e9))
+* Disable the request logging to avoid logging password and other request data (#287) ([6d8813e](https://github.com/block-core/blockcore/commit/6d8813e)), closes [#287](https://github.com/block-core/blockcore/issues/287)
+* Feature/upload artifact on workflow (#293) ([8aefa13](https://github.com/block-core/blockcore/commit/8aefa13)), closes [#293](https://github.com/block-core/blockcore/issues/293) [#291](https://github.com/block-core/blockcore/issues/291)
+* Launch setting names should use the technical chain name (#294) ([2f1b241](https://github.com/block-core/blockcore/commit/2f1b241)), closes [#294](https://github.com/block-core/blockcore/issues/294)
+* Resolve PR comments from initial PR on HomeCoin (#295) ([69dfb03](https://github.com/block-core/blockcore/commit/69dfb03)), closes [#295](https://github.com/block-core/blockcore/issues/295)
+
+
+
 ##  (2021-02-19)
 
 * Add checkpoints for City Chain network ([ef16368](https://github.com/block-core/blockcore/commit/ef16368))
diff --git a/src/Node/Blockcore.Node/NodeBuilder.cs b/src/Node/Blockcore.Node/NodeBuilder.cs
index 843461e3e..79362cfe5 100644
--- a/src/Node/Blockcore.Node/NodeBuilder.cs
+++ b/src/Node/Blockcore.Node/NodeBuilder.cs
@@ -34,27 +34,25 @@ public static IFullNodeBuilder Create(string chain, NodeSettings settings)
             switch (chain)
             {
                 case "BTC":
+                case "XRC":
                     nodeBuilder.UsePowConsensus().AddMining().UseWallet();
                     break;
-                case "X42":
-                    nodeBuilder.UsePosConsensus().AddPowPosMining().UseColdStakingWallet().UseWatchOnlyWallet();
-                    break;
                 case "X1":
-                    nodeBuilder.UseX1Consensus().UseColdStakingWallet(); ;
+                    nodeBuilder.UseX1Consensus().UseColdStakingWallet();
                     break;
+                case "AMS":
+                case "X42":
                 case "BCP":
                 case "CITY":
                 case "STRAT":
                 case "RUTA":
                 case "EXOS":
                 case "XDS":
+                case "XLR":
                 case "IMPLX":
                 case "HOME":
                     nodeBuilder.UsePosConsensus().AddPowPosMining().UseColdStakingWallet();
                     break;
-                case "XRC":
-                    nodeBuilder.UsePowConsensus().AddMining().UseWallet();
-                    break;
             }
 
             return nodeBuilder;