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: Bumped up exchanged value #2972

Merged
merged 2 commits into from
Sep 16, 2024
Merged

Conversation

ebadiere
Copy link
Collaborator

Bumped up exchanged value as the HTS token create will take the fee from the value instead of the gas.

Description:

Related issue(s):

Fixes #2951

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

fee from the value instead of the gas.

Signed-off-by: ebadiere <[email protected]>
@ebadiere ebadiere added the bug Something isn't working label Sep 12, 2024
@ebadiere ebadiere added this to the 0.56.0 milestone Sep 12, 2024
@ebadiere ebadiere self-assigned this Sep 12, 2024
Copy link

github-actions bot commented Sep 12, 2024

Tests

       3 files     274 suites   19s ⏱️
1 289 tests 1 288 ✔️ 1 💤 0
1 301 runs  1 300 ✔️ 1 💤 0

Results for commit 9f5ab65.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Sep 12, 2024

Acceptance Tests

  17 files  218 suites   29m 20s ⏱️
602 tests 597 ✔️ 4 💤 1
617 runs  612 ✔️ 4 💤 1

Results for commit 9f5ab65.

♻️ This comment has been updated with latest results.

quiet-node
quiet-node previously approved these changes Sep 16, 2024
acuarica
acuarica previously approved these changes Sep 16, 2024
Copy link
Contributor

@acuarica acuarica left a comment

Choose a reason for hiding this comment

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

lg, just a left a nit

packages/server/tests/acceptance/rpc_batch2.spec.ts Outdated Show resolved Hide resolved
@ebadiere ebadiere dismissed stale reviews from acuarica and quiet-node via 9f5ab65 September 16, 2024 17:01
Co-authored-by: Luis Mastrangelo <[email protected]>
Signed-off-by: Eric Badiere <[email protected]>
Copy link

sonarcloud bot commented Sep 16, 2024

Copy link

codecov bot commented Sep 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.54%. Comparing base (8245f51) to head (9f5ab65).
Report is 7 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2972   +/-   ##
=======================================
  Coverage   89.53%   89.54%           
=======================================
  Files          56       56           
  Lines        3794     3796    +2     
  Branches      772      775    +3     
=======================================
+ Hits         3397     3399    +2     
  Misses        350      350           
  Partials       47       47           
Flag Coverage Δ
relay 89.68% <ø> (+<0.01%) ⬆️
server 88.15% <ø> (ø)
ws-server 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1 file with indirect coverage changes

Copy link
Contributor

@acuarica acuarica left a comment

Choose a reason for hiding this comment

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

lg

@ebadiere ebadiere merged commit 3926aa9 into main Sep 16, 2024
39 checks passed
@ebadiere ebadiere deleted the 2951-TokenCreate-createNonFungibleToken branch September 16, 2024 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
3 participants