wallet: enable sending to silent payment address

src/addresstype.cpp

@@ -156,9 +156,7 @@ class ValidDestinationVisitor
     bool operator()(const PubKeyDestination& dest) const { return false; }
     bool operator()(const PKHash& dest) const { return true; }
     bool operator()(const ScriptHash& dest) const { return true; }
-    // silent payment addresses are not valid until sending support has been implemented
-    // TODO: set this to true once sending is implemented
-    bool operator()(const V0SilentPaymentDestination& dest) const { return false; }
+    bool operator()(const V0SilentPaymentDestination& dest) const { return true; }
     bool operator()(const WitnessV0KeyHash& dest) const { return true; }
     bool operator()(const WitnessV0ScriptHash& dest) const { return true; }
     bool operator()(const WitnessV1Taproot& dest) const { return true; }

src/key_io.cpp

@@ -173,11 +173,6 @@ CTxDestination DecodeDestination(const std::string& str, const CChainParams& par
             }
             CPubKey scan_pubkey{data.begin(), data.begin() + CPubKey::COMPRESSED_SIZE};
             CPubKey spend_pubkey{data.begin() + CPubKey::COMPRESSED_SIZE, data.begin() + 2*CPubKey::COMPRESSED_SIZE};
-            // This is a bit of a hack to disable silent payments until sending is implemented. The reason we return a V0SilentPaymentDestination
-            // while also setting an error message is so that we can use DecodeDestination in the unit tests, but also have `validateaddress` fail
-            // when passed a silent payment address
-            // TODO: remove this error_str once sending support is implemented
-            error_str = strprintf("This is a valid Silent Payments v0 address, but sending support is not yet implemented.");
             return V0SilentPaymentDestination{scan_pubkey, spend_pubkey};
         }
         // Bech32 decoding