Skip to content

Commit

Permalink
Revert to SignDeployContractTxParams and SignExecuteScriptTxParams wh…
Browse files Browse the repository at this point in the history
…en building tx params
  • Loading branch information
h0ngcha0 committed Oct 5, 2024
1 parent 4286900 commit 268de9d
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 14 deletions.
14 changes: 5 additions & 9 deletions packages/web3/src/contract/contract.ts
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,7 @@ import {
SignExecuteScriptTxParams,
SignerProvider,
Address,
SignExecuteScriptTxResult,
SignDeployContractChainedTxParams,
SignExecuteScriptChainedTxParams
SignExecuteScriptTxResult
} from '../signer'
import * as ralph from './ralph'
import {
Expand Down Expand Up @@ -567,7 +565,7 @@ export class Contract extends Artifact {
async txParamsForDeployment<P extends Fields>(
signer: SignerProvider,
params: DeployContractParams<P>
): Promise<SignDeployContractChainedTxParams> {
): Promise<SignDeployContractTxParams> {
const isDevnet = await this.isDevnet(signer)
const initialFields: Fields = params.initialFields ?? {}
const bytecode = this.buildByteCodeToDeploy(
Expand All @@ -576,8 +574,7 @@ export class Contract extends Artifact {
params.exposePrivateFunctions ?? false
)
const selectedAccount = await signer.getSelectedAccount()
const signerParams: SignDeployContractChainedTxParams = {
type: 'DeployContract',
const signerParams: SignDeployContractTxParams = {
signerAddress: selectedAccount.address,
signerKeyType: selectedAccount.keyType,
bytecode: bytecode,
Expand Down Expand Up @@ -755,10 +752,9 @@ export class Script extends Artifact {
async txParamsForExecution<P extends Fields>(
signer: SignerProvider,
params: ExecuteScriptParams<P>
): Promise<SignExecuteScriptChainedTxParams> {
): Promise<SignExecuteScriptTxParams> {
const selectedAccount = await signer.getSelectedAccount()
const signerParams: SignExecuteScriptChainedTxParams = {
type: 'ExecuteScript',
const signerParams: SignExecuteScriptTxParams = {
signerAddress: selectedAccount.address,
signerKeyType: selectedAccount.keyType,
bytecode: this.buildByteCodeToDeploy(params.initialFields ?? {}),
Expand Down
4 changes: 1 addition & 3 deletions packages/web3/src/signer/signer.ts
Original file line number Diff line number Diff line change
Expand Up @@ -40,9 +40,7 @@ import {
SubmissionResult,
SubmitTransactionParams,
KeyType,
MessageHasher,
SignChainedTxParams,
SignChainedTxResult
MessageHasher
} from './types'
import { TransactionBuilder } from './tx-builder'
import { addressFromPublicKey, groupOfAddress } from '../address'
Expand Down
4 changes: 2 additions & 2 deletions test/transaction.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -178,7 +178,7 @@ describe('transactions', function () {
}

const [transferResult, deployResult] = await TransactionBuilder.from(nodeProvider).buildChainedTx(
[transferTxParams, deployTxParams],
[transferTxParams, { ...deployTxParams, type: 'DeployContract' }],
[signer1.publicKey, signer2.publicKey]
)

Expand Down Expand Up @@ -240,7 +240,7 @@ describe('transactions', function () {
}

const [transferResult, depositResult] = await TransactionBuilder.from(nodeProvider).buildChainedTx(
[transferTxParams, depositTxParams],
[transferTxParams, { ...depositTxParams, type: 'ExecuteScript' }],
[signer1.publicKey, signer2.publicKey]
)

Expand Down

0 comments on commit 268de9d

Please sign in to comment.