Skip to content

Commit

Permalink
fix: add default sign options to wallet connect sign funcs (#73)
Browse files Browse the repository at this point in the history
This PR adds the `preferNoSetFee` and `preferNoSetMemo` options
hardcoded to `true` to the `WalletConnectV2` class. See #61 for the
relevant issue. If tested to be working, this PR should take precedence
over #71 as this implementation is much simpler.

- Closes #61
- Closes #71
  • Loading branch information
AaronCQL authored Jun 21, 2024
1 parent 0648f8a commit 36ef5e0
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions src/wallet/walletconnect/WalletConnectV2.ts
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,11 @@ const Event = {
} as const;
type Event = (typeof Event)[keyof typeof Event];

const DEFAULT_SIGN_OPTIONS = {
preferNoSetFee: true,
preferNoSetMemo: true,
};

export class WalletConnectV2 {
private readonly projectId: string;
private readonly mobileAppDetails: MobileAppDetails;
Expand Down Expand Up @@ -191,6 +196,7 @@ export class WalletConnectV2 {
{
signerAddress,
signDoc: stdSignDoc,
signOptions: DEFAULT_SIGN_OPTIONS,
}
);
return {
Expand All @@ -210,6 +216,7 @@ export class WalletConnectV2 {
{
signerAddress,
signDoc,
signOptions: DEFAULT_SIGN_OPTIONS,
}
);
return {
Expand Down

0 comments on commit 36ef5e0

Please sign in to comment.