onflow / onflow/flow-cli

[BUG] Extra argument passed to transactions in Stable Cadence Pre-release version

Open
#1,208 5 comments 0 reactions 1 assignee View on GitHub

@chasefleming is already working on this.

Since Sep 21, 2023.

Bug Feedback
Dominant language
Go
Stars
209
Forks
87
Avg merge
1d 17h
Merged PRs (30d)
3

Description

Problem

When trying to send a transaction with parameters, I consistently get

❌ Command Error: error parsing transaction arguments: argument count is [n+1], expected [n]
Steps to Reproduce

With the Stable Cadence CLI preview release installed, try the following transaction:

transaction(s: String) {
    prepare(signer: &Account) {
        log(s)
    }
}
flow transactions send ./transactions/test_param.cdc hello

And it should output:

❌ Command Error: error parsing transaction arguments: argument count is 2, expected 1

Confirm it works without the transaction parameter:

transaction {
    prepare(signer: &Account) {
        log("hello")
    }
}
flow transactions send ./transactions/test.cdc

And it should succeed.

Acceptance Criteria

Transactions with and without params should execute.

Context

Using the preview release to update FLIPs and contracts.

Machine Info

OS: macOS 13.6 (22G120)
CLI: v1.5.0-stable-cadence.2

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.