Merge pull request #1389 from PhilippeR26/Ledger231

feat: handling of Ledger Starknet APP v2.3.1

Author: tabaktoni

src/signer/index.ts

@@ -8,3 +8,4 @@ export {
   getLedgerPathBuffer111 as getLedgerPathBuffer,
 } from './ledgerSigner111';
 export { LedgerSigner221, getLedgerPathBuffer221 } from './ledgerSigner221';
+export { LedgerSigner231 } from './ledgerSigner231';

src/signer/ledgerSigner221.ts

@@ -134,7 +134,7 @@ export class LedgerSigner221<Transport extends Record<any, any> = any>
       const ledgerResponse = await this.signTxV1(det, transactions);
       assert(
         toBigInt(msgHash) === ledgerResponse.hash,
-        'The transaction hash calculated by Starknet.js is different from the one calculated by the Ledger.'
+        'The V1 transaction hash calculated by Starknet.js is different from the one calculated by the Ledger.'
       ); // probably non compatibility with Cairo 0
       return ledgerResponse.signature;
     }
@@ -151,7 +151,7 @@ export class LedgerSigner221<Transport extends Record<any, any> = any>
       const ledgerResponse = await this.signTxV3(det, transactions);
       assert(
         toBigInt(msgHash) === ledgerResponse.hash,
-        'The transaction hash calculated by Starknet.js is different from the one calculated by the Ledger.'
+        'The V3 transaction hash calculated by Starknet.js is different from the one calculated by the Ledger.'
       ); // probably non compatibility with Cairo 0
       return ledgerResponse.signature;
     }