-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
ci(release): v0.95.0
@ master
#3269
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
fuel-service-user
requested review from
digorithm,
arboleya,
Torres-ssf,
Dhaiwat10,
danielbate,
nedsalk,
petertonysmith94 and
maschad
as code owners
October 7, 2024 08:41
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
CodSpeed Performance ReportMerging #3269 will not alter performanceComparing Summary
|
fuel-service-user
force-pushed
the
changeset-release/master
branch
from
October 7, 2024 12:31
75e8f31
to
e23f182
Compare
fuel-service-user
force-pushed
the
changeset-release/master
branch
from
October 7, 2024 12:52
e23f182
to
e73f597
Compare
fuel-service-user
force-pushed
the
changeset-release/master
branch
2 times, most recently
from
October 7, 2024 16:06
3abfa80
to
10091cb
Compare
arboleya
approved these changes
Oct 7, 2024
nedsalk
previously approved these changes
Oct 7, 2024
petertonysmith94
previously approved these changes
Oct 7, 2024
arboleya
previously approved these changes
Oct 7, 2024
fuel-service-user
dismissed stale reviews from arboleya, petertonysmith94, and nedsalk
via
October 8, 2024 12:52
776f34a
fuel-service-user
force-pushed
the
changeset-release/master
branch
from
October 10, 2024 19:29
98de133
to
8e877ce
Compare
@FuelLabs/sdk-ts Should these be identified as breaking?
|
fuel-service-user
force-pushed
the
changeset-release/master
branch
from
October 10, 2024 19:50
8e877ce
to
b234e99
Compare
petertonysmith94
previously approved these changes
Oct 10, 2024
arboleya
previously approved these changes
Oct 10, 2024
maschad
previously approved these changes
Oct 10, 2024
Torres-ssf
previously approved these changes
Oct 10, 2024
fuel-service-user
dismissed stale reviews from Torres-ssf, petertonysmith94, maschad, and arboleya
via
October 10, 2024 21:04
b171f0e
fuel-service-user
force-pushed
the
changeset-release/master
branch
from
October 10, 2024 21:04
db827e4
to
b171f0e
Compare
Torres-ssf
approved these changes
Oct 10, 2024
arboleya
approved these changes
Oct 10, 2024
maschad
approved these changes
Oct 10, 2024
Coverage Report:
Changed Files:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
In this release, we:
checksum
utility to theAddress
classisUserAccount
andgetAddressType
which indicate the type of the hex passedfuels
version usedprovider.getTransactions()
method.0
in thebn
class returned an incorrect valueProvider
.ContractFactory
[email protected]
[email protected]
create-fuels
toolchain file[email protected]
Address.toString
andAddress.valueOf
returns the Address checksumfuel-core
to0.38.0
getBlockWithTransactions
queryBreaking
toString
andvalueOf
returns checksum, by @Torres-ssfchainInfoFragment
andGasCostsFragment
, by @nedsalkFeatures
getTransactions
, by @petertonysmith94Fixes
bn
formatting bug with 0 units, by @maschadChores
[email protected]
,[email protected]
, andcreate-fuels
toolchains, by @danielbate[email protected]
, by @nedsalkchain
andnodeInfo
queries into one, by @nedsalkfuel-core
to0.38.0
, by @Dhaiwat10Migration Notes
Features
#3306 - Bump transaction pagination limit to 60
provider.getTransactions()
method.Chores
#3310 - Made Address
toString
andvalueOf
returns checksumThe return of both
Address.toString()
andAddress.valueOf
was modified to return the address checksum instead of the Bech32 string#3286 - Slim down
chainInfoFragment
andGasCostsFragment
latestBlock
is no longer part of theChainInfo
return ofprovider.getChain()
. You can fetch it viaprovider.getBlock('latest')
.ChainInfo['consensusParameters']['gasCosts']
has been slimmed down to only contain data necessary for the operation of the SDK. Up until now, the SDK was fetching more than it needed. If this change affects you, you will have to create a custom graphql query forgasCosts
for the additional data you need.#3296 - Optimize balance queries
owner
andassetId
properties from the response ofProvider.operations.getBalance()
. These properties are also required arguments to execute the function so are redundant in the response. Should you require these values, you should take them from the values that you passed to the function.owner
property from the response ofProvider.operations.getBalances()
. This property is a required argument to execute the function so is redundant in the response. Should you require this value, you should take it from the value that you passed to the function.