RequestSignTransactionNimiq,RequestSignMessageNimiq: require 2.0 app if needed

For message signing and Albatross transactions require app version 2.0, as support for
these was added in that app version.

Author: danimoh

src/high-level-api/requests/bitcoin/request-sign-transaction-bitcoin.ts

@@ -73,6 +73,20 @@ export default class RequestSignTransactionBitcoin extends RequestBitcoin<string
     public readonly network: Exclude<Network, Network.DEVNET>;
     private _inputType: AddressTypeBitcoin;
 
+    public get requiredApp(): string {
+        // The new Bitcoin app does not allow custom input scripts (for example custom witness scripts) unless an
+        // associated wallet policy had been registered. For this reason, we currently still require using the old api
+        // if custom input scripts are set.
+        if (!this.transaction.inputs.some(({ customScript }) => !!customScript)) return super.requiredApp;
+        if (this._coinAppConnection && this._coinAppConnection.app === super.requiredApp
+            && !RequestBitcoin._isNewApiSupported(this._coinAppConnection.app, this._coinAppConnection.appVersion)) {
+            // We're already connected to an appropriate Bitcoin app on which we don't use the new api, e.g. a Bitcoin
+            // app before 2.0. Thus, no need to specifically require the Legacy app variant.
+            return super.requiredApp;
+        }
+        return getLegacyApp(super.requiredApp); // require Legacy app variant
+    }
+
     constructor(transaction: TransactionInfoBitcoin, expectedWalletId?: string) {
         super(expectedWalletId);
 
@@ -145,20 +159,6 @@ export default class RequestSignTransactionBitcoin extends RequestBitcoin<string
         this._loadBitcoinLibIfNeeded().catch(() => {});
     }
 
-    public get requiredApp(): string {
-        // The new Bitcoin app does not allow custom input scripts (for example custom witness scripts) unless an
-        // associated wallet policy had been registered. For this reason, we currently still require using the old api
-        // if custom input scripts are set.
-        if (!this.transaction.inputs.some(({ customScript }) => !!customScript)) return super.requiredApp;
-        if (this._coinAppConnection && this._coinAppConnection.app === super.requiredApp
-            && !RequestBitcoin._isNewApiSupported(this._coinAppConnection.app, this._coinAppConnection.appVersion)) {
-            // We're already connected to an appropriate Bitcoin app on which we don't use the new api, e.g. a Bitcoin
-            // app before 2.0. Thus, no need to specifically require the Legacy app variant.
-            return super.requiredApp;
-        }
-        return getLegacyApp(super.requiredApp); // require Legacy app variant
-    }
-
     public async call(transport: Transport): Promise<string> {
         // Resources:
         // - to learn more about scripts and how input and output script relate to each other:

src/high-level-api/requests/nimiq/request-nimiq.ts

@@ -18,9 +18,12 @@ export default abstract class RequestNimiq<Version extends NimiqVersion, T>
 
     public readonly coin: Coin.NIMIQ = Coin.NIMIQ;
     public readonly requiredApp: string = 'Nimiq';
-    public readonly minRequiredAppVersion: string = '1.4.2'; // first version supporting web usb
     public readonly nimiqVersion: Version;
 
+    public get minRequiredAppVersion(): string {
+        return '1.4.2'; // first version supporting web usb
+    }
+
     protected constructor(nimiqVersion: Version, expectedWalletId?: string) {
         super(expectedWalletId);
         this.nimiqVersion = nimiqVersion;

src/high-level-api/requests/nimiq/request-sign-message-nimiq.ts

@@ -19,6 +19,10 @@ export default class RequestSignMessageNimiq<Version extends NimiqVersion>
         preferDisplayTypeHash: boolean, // second choice, if multiple flags are set
     };
 
+    public get minRequiredAppVersion(): string {
+        return '2.0'; // first version supporting message signing
+    }
+
     constructor(
         nimiqVersion: Version,
         keyPath: string,

src/high-level-api/requests/nimiq/request-sign-transaction-nimiq.ts

@@ -36,6 +36,12 @@ export default class RequestSignTransactionNimiq<Version extends NimiqVersion>
     public readonly type: RequestTypeNimiq.SIGN_TRANSACTION;
     public readonly transaction: TransactionInfoNimiq<Version>;
 
+    public get minRequiredAppVersion(): string {
+        return this.nimiqVersion === NimiqVersion.ALBATROSS
+            ? '2.0' // first version supporting Albatross transactions
+            : super.minRequiredAppVersion;
+    }
+
     constructor(
         nimiqVersion: Version,
         keyPath: string,