Skip to content

Commit

Permalink
fix: use BigNumberish type for classHash
Browse files Browse the repository at this point in the history
  • Loading branch information
dhruvkelawala committed Oct 10, 2022
1 parent 48817d6 commit 0ab4d47
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion src/types/lib.ts
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ export type DeployContractPayload = {

export type DeclareContractPayload = {
contract: CompiledContract | string;
classHash: string; // Once the classHash is included in CompiledContract, this can be removedf
classHash: BigNumberish; // Once the classHash is included in CompiledContract, this can be removedf
};

export type DeclareContractTransaction = {
Expand Down
2 changes: 1 addition & 1 deletion src/types/signer.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ export interface InvocationsSignerDetails extends Required<InvocationsDetails> {

export interface DeclareSignerDetails {
// contractClass: ContractClass, // Should be used once class hash is present in ContractClass
classHash: string;
classHash: BigNumberish;
senderAddress: BigNumberish;
chainId: StarknetChainId;
maxFee: BigNumberish;
Expand Down
2 changes: 1 addition & 1 deletion src/utils/hash.ts
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,7 @@ export function calculateDeployTransactionHash(

export function calculateDeclareTransactionHash(
// contractClass: ContractClass, // Should be used once class hash is present in ContractClass
classHash: string,
classHash: BigNumberish,
senderAddress: BigNumberish,
version: BigNumberish,
maxFee: BigNumberish,
Expand Down

0 comments on commit 0ab4d47

Please sign in to comment.