Skip to content
This repository has been archived by the owner on Oct 4, 2024. It is now read-only.

Remove required fields check from Deploy Account tx #407

Merged
merged 1 commit into from
Jan 19, 2024

Conversation

pefontana
Copy link
Member

Remove required fields check from Deploy Account tx

[error] GenServer :testnet_state_sync terminating
** (Ecto.InvalidChangesetError) could not perform insert because changeset is invalid.

Changeset

    #Ecto.Changeset<
      action: :insert,
      changes: %{
        class_hash: "0x2fadbf77a721b94bdcc3032d86a8921661717fa55145bccf88160ee2a5efcd1",
        constructor_calldata: ["0x74c169c5cda8d5f205198c432381cfa71eff52f45fbff18d5a243a0be64b8f0",
         "0x0"],
        contract_address_salt: "0x74c169c5cda8d5f205198c432381cfa71eff52f45fbff18d5a243a0be64b8f0",
        hash: "0x3e62bb89efc8d1af707d2e1bec1fd72c0bc03f7f13a45afbd126c06761e442d",
        network: :testnet,
        nonce: "0x0",
        signature: ["0x5f515f16d4b9251e4d4420a11ffe9ef4cae759228b5a3898a0de6ce72a1d123",
         "0x4eddbe1993546a5dd9ff2eb8eca88a7e7f827eec837ede65ab2b2dff19037c"],
        type: "DEPLOY_ACCOUNT",
        version: "0x3"
      },
      errors: [max_fee: {"can't be blank", [validation: :required]}],
      data: #StarknetExplorer.Transaction<>,
      valid?: false

@pefontana pefontana added this pull request to the merge queue Jan 19, 2024
Merged via the queue into main with commit 36beaca Jan 19, 2024
1 check passed
@pefontana pefontana deleted the remove-required-fields-from-deploy-account branch January 19, 2024 14:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants