Skip to content

Commit

Permalink
Merge pull request #807 from airgap-it/fix/getPKH_rename
Browse files Browse the repository at this point in the history
fix: rename
  • Loading branch information
jsamol authored Sep 4, 2024
2 parents d0e7c59 + 89fd473 commit 5caa59a
Showing 1 changed file with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -383,7 +383,7 @@ export class WalletConnectCommunicationClient extends CommunicationClient {
throw new MissingRequiredScope(PermissionScopeMethods.SIGN)
}
const network = this.getActiveNetwork()
const account = await this.getPKH()
const account = await this.getAccountOrPK()
this.validateNetworkAndAccount(network, account)

this.checkWalletReadiness(this.getTopicFromSession(session))
Expand Down Expand Up @@ -445,7 +445,7 @@ export class WalletConnectCommunicationClient extends CommunicationClient {
throw new MissingRequiredScope(PermissionScopeMethods.OPERATION_REQUEST)
}
const network = this.getActiveNetwork()
const account = await this.getPKH()
const account = await this.getAccountOrPK()
this.validateNetworkAndAccount(network, account)
this.checkWalletReadiness(this.getTopicFromSession(session))

Expand Down Expand Up @@ -1330,10 +1330,10 @@ export class WalletConnectCommunicationClient extends CommunicationClient {
}

/**
* @description Access the public key hash of the active account
* @description Access the public key hash (or public key) of the active account
* @error ActiveAccountUnspecified thrown when there are multiple Tezos account in the session and none is set as the active one
*/
async getPKH() {
async getAccountOrPK() {
if (!this.activeAccountOrPbk) {
this.getSession()
throw new ActiveAccountUnspecified()
Expand Down

0 comments on commit 5caa59a

Please sign in to comment.