fuels-ts icon indicating copy to clipboard operation
fuels-ts copied to clipboard

Published 20 hours ago • verified publisher icon FuelLabs

Method `assembleTransactionSummary` returns incorrect data

Open matt-user opened this issue 1 year ago • 1 comments

What version of fuels-ts are you using?

0.71.1

Steps to Reproduce

Use assembleTransactionSummary and do not pass in the optional gqlTransactionStatus argument.

Expected Behavior

isStatusFailure, isStatusSuccess, and isStatusPending should return correctly based on the status of the transaction and its receipts

Actual Behavior

isStatusFailure, isStatusSuccess, and isStatusPending always return false if gqlTransactionStatus is not passed in.

matt-user avatar Jan 22 '24 17:01 matt-user

Related:

  • https://github.com/FuelLabs/fuels-ts/issues/1578

arboleya avatar Mar 22 '24 16:03 arboleya

Could you elaborate a bit more on the use case @matt-user ? I'm curious because the getTransactionSummary method can be used to assemble a TransactionSummary can be used to see the transaction's status, indeed the the processGraphqlStatus relies on the gqlTransactionStatus to return these values but that seems like expected behaviour.

maschad avatar Apr 05 '24 15:04 maschad

@maschad hmm yeah that makes sense. It's been a while since I made this issue. Let me look in our frontend code and get back to you.

matt-user avatar Apr 17 '24 15:04 matt-user

Yeah I think we can close this issue. We ran into this because of a workaround which has now been fixed on the sdk side. https://github.com/FuelLabs/fuels-wallet/issues/1236

matt-user avatar Apr 17 '24 15:04 matt-user