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

Fix: remove dead gas estimation method arguments, which were breaking the flow #1445

Merged
merged 2 commits into from
Apr 2, 2024

Conversation

YarikRevich
Copy link
Collaborator

No description provided.

@YarikRevich YarikRevich self-assigned this Mar 30, 2024
@YarikRevich YarikRevich added the enhancement New feature or request label Mar 30, 2024
Copy link
Member

@dabasov dabasov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@YarikRevich YarikRevich changed the title Hotfix: remove dead gas estimation method arguments, which were breaking the flow Fix: remove dead gas estimation method arguments, which were breaking the flow Apr 2, 2024
@YarikRevich YarikRevich merged commit 3c834f4 into sprint-1.14 Apr 2, 2024
8 of 9 checks passed
@YarikRevich YarikRevich deleted the hotfix/gas-estimation branch April 2, 2024 17:40
peterlimg added a commit that referenced this pull request Apr 12, 2024
* sprint-1.14: (72 commits)
  fix broken pipe error
  Fix
  Fix
  fix remaining size
  parallel hashing for high upload mode
  fix build issue
  refactor
  get client state fields in getWalletBalance wasm api
  fix: removed dead method arguments, which were breaking the flow (#1445)
  add upload modes
  Update go version
  Update tests.yml
  updated golint version latest tests.yml
  updated node version
  updated go version
  fix: fixed bugs
  fix: fixed bugs
  fix: fixed bugs
  fix: fixed bugs
  feature: implemented gas amount estimation
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants