Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: introduce node command #2669

Merged
merged 49 commits into from
Aug 22, 2022
Merged

feat: introduce node command #2669

merged 49 commits into from
Aug 22, 2022

Commits on Jul 29, 2022

  1. add node command with bank query and send

    Bounty address: cosmos1zra3wz596n0qc898myceka3q23x9rk8fuw65c7
    gjermundgaraba authored and tbruyelle committed Jul 29, 2022
    Configuration menu
    Copy the full SHA
    168aeb4 View commit details
    Browse the repository at this point in the history
  2. Code review cleanup

    gjermundgaraba authored and tbruyelle committed Jul 29, 2022
    Configuration menu
    Copy the full SHA
    04946df View commit details
    Browse the repository at this point in the history
  3. Code review fix

    gjermundgaraba authored and tbruyelle committed Jul 29, 2022
    Configuration menu
    Copy the full SHA
    7727145 View commit details
    Browse the repository at this point in the history
  4. fix merge

    ilgooz authored and tbruyelle committed Jul 29, 2022
    Configuration menu
    Copy the full SHA
    94452af View commit details
    Browse the repository at this point in the history
  5. refactor

    ilgooz authored and tbruyelle committed Jul 29, 2022
    Configuration menu
    Copy the full SHA
    4577143 View commit details
    Browse the repository at this point in the history
  6. Merge branch 'develop' into issue-2489-node-banks-cmds

    Fixed a bunch of conflicts with recent renaming
    tbruyelle committed Jul 29, 2022
    Configuration menu
    Copy the full SHA
    8924f77 View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    c4508b1 View commit details
    Browse the repository at this point in the history
  8. Configuration menu
    Copy the full SHA
    a3760f8 View commit details
    Browse the repository at this point in the history
  9. Configuration menu
    Copy the full SHA
    09ce61b View commit details
    Browse the repository at this point in the history
  10. Configuration menu
    Copy the full SHA
    92b9ece View commit details
    Browse the repository at this point in the history
  11. test: read test.v to add more trace on step execution

    That helps when some commands are not running properly.
    tbruyelle committed Jul 29, 2022
    Configuration menu
    Copy the full SHA
    8e39bda View commit details
    Browse the repository at this point in the history
  12. Configuration menu
    Copy the full SHA
    c66131f View commit details
    Browse the repository at this point in the history
  13. Ensure bank send can take account name or address

    Also simplify the logic by removing the read of the --from flag, which
    is no longer used in this command.
    
    BroadcastTx now takes a cosmosaccount.Account instead of just an account
    name. Since ensure the account comes from the keyring, and avoid the
    BroadcastTx impl to re-check that (because the keyring is always
    checked before the call to BroadcastTx).
    tbruyelle committed Jul 29, 2022
    Configuration menu
    Copy the full SHA
    e736ce3 View commit details
    Browse the repository at this point in the history
  14. fix after rebase

    tbruyelle committed Jul 29, 2022
    Configuration menu
    Copy the full SHA
    fa89501 View commit details
    Browse the repository at this point in the history
  15. Configuration menu
    Copy the full SHA
    2c38d0d View commit details
    Browse the repository at this point in the history
  16. Configuration menu
    Copy the full SHA
    6fe217f View commit details
    Browse the repository at this point in the history
  17. add fees flag to node tx bank send cmd

    fees are required, at least in cosmos-hub to send funds to an other
    account. W/o fees the transaction returns this message:
    error code: '13' msg: 'insufficient fees; got:  required: 8uatom:
    insufficient fee'
    
    Also increase the additional gas amount added to the simulated gas,
    because I got some insufficient gas error with the previous value.
    tbruyelle committed Jul 29, 2022
    Configuration menu
    Copy the full SHA
    cb4ab7c View commit details
    Browse the repository at this point in the history
  18. Change tx BroadcastMode to BroadcastSync

    Previous mode BroadcastBlock alweus triggered a timeout error, even if
    the tx was finally accepted in a block.
    
    RPC error -32603 - Internal error: timed out waiting for tx to be
    included in a block
    tbruyelle committed Jul 29, 2022
    Configuration menu
    Copy the full SHA
    acc0827 View commit details
    Browse the repository at this point in the history
  19. fix linter

    tbruyelle committed Jul 29, 2022
    Configuration menu
    Copy the full SHA
    9f0901f View commit details
    Browse the repository at this point in the history
  20. fix other lint

    tbruyelle committed Jul 29, 2022
    Configuration menu
    Copy the full SHA
    cba7268 View commit details
    Browse the repository at this point in the history

Commits on Aug 1, 2022

  1. Configuration menu
    Copy the full SHA
    ffc26b5 View commit details
    Browse the repository at this point in the history
  2. fix typo

    tbruyelle committed Aug 1, 2022
    Configuration menu
    Copy the full SHA
    6b73579 View commit details
    Browse the repository at this point in the history

Commits on Aug 7, 2022

  1. wip

    tbruyelle committed Aug 7, 2022
    Configuration menu
    Copy the full SHA
    d06f79b View commit details
    Browse the repository at this point in the history
  2. simplify

    tbruyelle committed Aug 7, 2022
    Configuration menu
    Copy the full SHA
    ba95a64 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    d54f7d2 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    5d78357 View commit details
    Browse the repository at this point in the history

Commits on Aug 8, 2022

  1. Configuration menu
    Copy the full SHA
    3133230 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    6590d91 View commit details
    Browse the repository at this point in the history
  3. gofumpt

    tbruyelle committed Aug 8, 2022
    Configuration menu
    Copy the full SHA
    510ffa3 View commit details
    Browse the repository at this point in the history
  4. fix integration test

    tbruyelle committed Aug 8, 2022
    Configuration menu
    Copy the full SHA
    bc80125 View commit details
    Browse the repository at this point in the history

Commits on Aug 9, 2022

  1. Configuration menu
    Copy the full SHA
    b6fe5f9 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    497644c View commit details
    Browse the repository at this point in the history
  3. simplify node query

    tbruyelle committed Aug 9, 2022
    Configuration menu
    Copy the full SHA
    6be67e3 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    ea35e75 View commit details
    Browse the repository at this point in the history

Commits on Aug 10, 2022

  1. Configuration menu
    Copy the full SHA
    f8b63f0 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    89a0349 View commit details
    Browse the repository at this point in the history

Commits on Aug 11, 2022

  1. feat: add --broadcasd-mode flag to node tx command

    The default value is still block but the flag can change to sync in
    order to avoid timeout when the tx is broadcasted to busy nodes.
    
    Also add a `node query tx` so the user can check when his tx is
    included in a block.
    tbruyelle committed Aug 11, 2022
    Configuration menu
    Copy the full SHA
    e3b92cf View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    4d2ff28 View commit details
    Browse the repository at this point in the history
  3. comments

    tbruyelle committed Aug 11, 2022
    Configuration menu
    Copy the full SHA
    c1aabe2 View commit details
    Browse the repository at this point in the history

Commits on Aug 12, 2022

  1. Configuration menu
    Copy the full SHA
    350099c View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    a9b9f9b View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    10bd29e View commit details
    Browse the repository at this point in the history

Commits on Aug 16, 2022

  1. merge const decls

    tbruyelle committed Aug 16, 2022
    Configuration menu
    Copy the full SHA
    99b7748 View commit details
    Browse the repository at this point in the history
  2. Merge branch 'develop' into issue-2489-node-bank-cmds

    Alex Johnson committed Aug 16, 2022
    Configuration menu
    Copy the full SHA
    6c2e603 View commit details
    Browse the repository at this point in the history
  3. fix merge error

    aljo242 committed Aug 16, 2022
    Configuration menu
    Copy the full SHA
    3fa6677 View commit details
    Browse the repository at this point in the history

Commits on Aug 17, 2022

  1. Configuration menu
    Copy the full SHA
    7688a81 View commit details
    Browse the repository at this point in the history
  2. fix after merge

    tbruyelle committed Aug 17, 2022
    Configuration menu
    Copy the full SHA
    e8c0849 View commit details
    Browse the repository at this point in the history
  3. fix after merge

    tbruyelle committed Aug 17, 2022
    Configuration menu
    Copy the full SHA
    daebe1d View commit details
    Browse the repository at this point in the history

Commits on Aug 22, 2022

  1. Configuration menu
    Copy the full SHA
    8af2605 View commit details
    Browse the repository at this point in the history