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

Validate the value of the arguments before sending the transaction #840

Merged
merged 12 commits into from
Jul 18, 2022

Conversation

osipov-mit
Copy link
Member

@osipov-mit osipov-mit commented Jul 18, 2022

Changes

  • Rename api.program.is method to api.program.exists
    (@nikitayutanov @Pinkdesu if it used somewhere pls update it)
  • Add validation of gasLimit, value and codeId before sending transactions
  • api.code.submit method is now async
    (@Pinkdesu pls update it on frontend)
  • Add getting some runtime constants (existentialDeposit, blockGasLimit, mailboxTreshold, waitlistCost)
  • Add eslint config

Bump @gear-js/api version to 0.23.4

@osipov-mit osipov-mit added the api label Jul 18, 2022
api/src/Code.ts Outdated Show resolved Hide resolved
api/src/Program.ts Outdated Show resolved Hide resolved
api/src/utils/validate.ts Outdated Show resolved Hide resolved
api/src/Code.ts Outdated Show resolved Hide resolved
@osipov-mit osipov-mit requested a review from Pinkdesu July 18, 2022 10:45
@osipov-mit osipov-mit merged commit 56d556b into master Jul 18, 2022
@osipov-mit osipov-mit deleted the do-restrict-send-transactions branch July 18, 2022 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants