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: transferToContract method now allows big numbers #3458

Merged
merged 4 commits into from
Dec 9, 2024

Conversation

petertonysmith94
Copy link
Contributor

@petertonysmith94 petertonysmith94 commented Dec 8, 2024

Release notes

In this release, we:

  • Fixed transferToContract and batchTransferToContracts to accept big number amounts.

Checklist

  • All changes are covered by tests (or not applicable)
  • All changes are documented (or not applicable)
  • I reviewed the entire PR myself (preferably, on GH UI)
  • I described all Breaking Changes (or there's none)

@petertonysmith94 petertonysmith94 added the bug Issue is a bug label Dec 8, 2024
@petertonysmith94 petertonysmith94 self-assigned this Dec 8, 2024
Copy link

vercel bot commented Dec 8, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
fuels-template ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 9, 2024 9:49am
ts-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 9, 2024 9:49am
ts-docs-api ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 9, 2024 9:49am

Copy link

codspeed-hq bot commented Dec 8, 2024

CodSpeed Performance Report

Merging #3458 will not alter performance

Comparing ps/fix/transfer-to-contract-big-amounts (c249d44) with master (24edd5d)

Summary

✅ 18 untouched benchmarks

Copy link
Member

@Dhaiwat10 Dhaiwat10 left a comment

Choose a reason for hiding this comment

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

Can we add a test or two explicitly covering this addition?

Copy link
Contributor

github-actions bot commented Dec 9, 2024

Coverage Report:

Lines Branches Functions Statements
76.42%(+0%) 69.94%(+0%) 74.46%(+0%) 76.45%(+0%)
Changed Files:

Coverage values did not change👌.

@petertonysmith94 petertonysmith94 enabled auto-merge (squash) December 9, 2024 11:45
@petertonysmith94 petertonysmith94 merged commit 7390114 into master Dec 9, 2024
25 checks passed
@petertonysmith94 petertonysmith94 deleted the ps/fix/transfer-to-contract-big-amounts branch December 9, 2024 12:15
starc007 pushed a commit to starc007/fuels-ts that referenced this pull request Dec 9, 2024
* fix: incorrectly converting BN to number for `formatTransferToContractScriptData`

* chore: changeset

* chore: tests

* chore: added test for batch transfer
petertonysmith94 added a commit that referenced this pull request Dec 10, 2024
* feat: integrate vitest matchers globally

* fix: added global type file & changed setup file directory

* docs: improve `cookbook/transaction-request` (#3440)

* chore: silence recipes format logs (#3451)

* chore: silence recipe logs

* chore: update changeset

* docs: fix api links (#3452)

* docs: fixing API links

* chore: fixing relative links

* fix: `transferToContract` method now allows big numbers (#3458)

* fix: incorrectly converting BN to number for `formatTransferToContractScriptData`

* chore: changeset

* chore: tests

* chore: added test for batch transfer

* fix: moved vitest matcher setup from fuel-gauge to utils

* fix(utils): update vitest matcher import path

* chore: add changeset for vitest matcher changes

* fix: removed previous changeset

* chore: removed types

* chore: added changeset

* Update .changeset/rare-hornets-rush.md

---------

Co-authored-by: Sérgio Torres <30977845+Torres-ssf@users.noreply.github.com>
Co-authored-by: Daniel Bate <djbate23@gmail.com>
Co-authored-by: Peter Smith <peter@blueoceancomputing.co.uk>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue is a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

transferToContract not allows for big amounts
5 participants