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

Problem: GetEthBalance blocks the GameThread #270

Open
damoncro opened this issue Mar 2, 2023 · 6 comments
Open

Problem: GetEthBalance blocks the GameThread #270

damoncro opened this issue Mar 2, 2023 · 6 comments
Assignees
Labels
good first issue Good for newcomers

Comments

@damoncro
Copy link
Contributor

damoncro commented Mar 2, 2023

If can not get the balance (Timeout or whatever), the blocking may take 10+ seconds.

@damoncro
Copy link
Contributor Author

damoncro commented Mar 2, 2023

One of the errors:
LogTemp: Error: Can not get CRO balance, Error: CronosPlayUnreal GetEthAddress Error: Failed to obtain an account balance

@damoncro
Copy link
Contributor Author

damoncro commented Mar 7, 2023

Due to crypto-com/defi-wallet-core-rs#858

@leejw51crypto
Copy link
Collaborator

change to async using AsyncTask

@leejw51crypto leejw51crypto self-assigned this Mar 20, 2023
@leejw51crypto leejw51crypto added the good first issue Good for newcomers label Mar 20, 2023
@leejw51crypto
Copy link
Collaborator

changed sync -> async

@leejw51crypto
Copy link
Collaborator

checking other functions now

@leejw51crypto
Copy link
Collaborator

almost changed to async
working on sendamount to async

leejw51crypto added a commit to leejw51crypto/play-unreal-plugin that referenced this issue Mar 23, 2023
Solution: convert sync apis to async

change GetEthBalance to async

change GetBalance to async

convert GetNFTSupply to async

convert GetNFTOwner to async

convert GetNFTCollection to async

convert GetNFTDenom to async

change GetNFTDenomByName to async

convert GetNFTAllDenoms to async

refactor sendamount

make SendAmount to async
leejw51crypto added a commit that referenced this issue Mar 23, 2023
Solution: convert sync apis to async

change GetEthBalance to async

change GetBalance to async

convert GetNFTSupply to async

convert GetNFTOwner to async

convert GetNFTCollection to async

convert GetNFTDenom to async

change GetNFTDenomByName to async

convert GetNFTAllDenoms to async

refactor sendamount

make SendAmount to async

convert GetNFTToken to async

convert Erc20Balance to async

convert Erc721Balance to async

convert Erc1155Balance to async

convert Erc1155BalanceOfBatch to async

convert Erc721Name to async

convert Erc721Symbol to async

convert Erc721Uri to async

convert Erc721GetApproved to async

convert Erc721IsApprovedForAll to async

convert Erc1155Uri to async

convert Erc1155IsApprovedForAll to async

convert Erc721Owner to async

convert Erc721TotalSupply to async

convert Erc721TokenByIndex to async

convert Erc721TokenOwnerByIndex to async

convert Erc20Name to async

make Erc20Symbol to async

make Erc20Decimals to async

make Erc20TotalSupply to async

make Erc20Allowance to async

tidy up comment
leejw51crypto added a commit to leejw51crypto/play-unreal-plugin that referenced this issue Mar 23, 2023
Solution: convert sync apis to async

change GetEthBalance to async

change GetBalance to async

convert GetNFTSupply to async

convert GetNFTOwner to async

convert GetNFTCollection to async

convert GetNFTDenom to async

change GetNFTDenomByName to async

convert GetNFTAllDenoms to async

refactor sendamount

make SendAmount to async

convert GetNFTToken to async

convert Erc20Balance to async

convert Erc721Balance to async

convert Erc1155Balance to async

convert Erc1155BalanceOfBatch to async

convert Erc721Name to async

convert Erc721Symbol to async

convert Erc721Uri to async

convert Erc721GetApproved to async

convert Erc721IsApprovedForAll to async

convert Erc1155Uri to async

convert Erc1155IsApprovedForAll to async

convert Erc721Owner to async

convert Erc721TotalSupply to async

convert Erc721TokenByIndex to async

convert Erc721TokenOwnerByIndex to async

convert Erc20Name to async

make Erc20Symbol to async

make Erc20Decimals to async

make Erc20TotalSupply to async

make Erc20Allowance to async

tidy up comment
leejw51crypto added a commit to leejw51crypto/play-unreal-plugin that referenced this issue Mar 28, 2023
Solution: convert sync apis to async

change GetEthBalance to async

change GetBalance to async

convert GetNFTSupply to async

convert GetNFTOwner to async

convert GetNFTCollection to async

convert GetNFTDenom to async

change GetNFTDenomByName to async

convert GetNFTAllDenoms to async

refactor sendamount

make SendAmount to async

convert GetNFTToken to async

convert Erc20Balance to async

convert Erc721Balance to async

convert Erc1155Balance to async

convert Erc1155BalanceOfBatch to async

convert Erc721Name to async

convert Erc721Symbol to async

convert Erc721Uri to async

convert Erc721GetApproved to async

convert Erc721IsApprovedForAll to async

convert Erc1155Uri to async

convert Erc1155IsApprovedForAll to async

convert Erc721Owner to async

convert Erc721TotalSupply to async

convert Erc721TokenByIndex to async

convert Erc721TokenOwnerByIndex to async

convert Erc20Name to async

make Erc20Symbol to async

make Erc20Decimals to async

make Erc20TotalSupply to async

make Erc20Allowance to async

tidy up comment
leejw51crypto added a commit to leejw51crypto/play-unreal-plugin that referenced this issue Mar 28, 2023
Solution: convert sync apis to async

change GetEthBalance to async

change GetBalance to async

convert GetNFTSupply to async

convert GetNFTOwner to async

convert GetNFTCollection to async

convert GetNFTDenom to async

change GetNFTDenomByName to async

convert GetNFTAllDenoms to async

refactor sendamount

make SendAmount to async

convert GetNFTToken to async

convert Erc20Balance to async

convert Erc721Balance to async

convert Erc1155Balance to async

convert Erc1155BalanceOfBatch to async

convert Erc721Name to async

convert Erc721Symbol to async

convert Erc721Uri to async

convert Erc721GetApproved to async

convert Erc721IsApprovedForAll to async

convert Erc1155Uri to async

convert Erc1155IsApprovedForAll to async

convert Erc721Owner to async

convert Erc721TotalSupply to async

convert Erc721TokenByIndex to async

convert Erc721TokenOwnerByIndex to async

convert Erc20Name to async

make Erc20Symbol to async

make Erc20Decimals to async

make Erc20TotalSupply to async

make Erc20Allowance to async

tidy up comment

fix uint64 overflow

fix compile

remove legacy

change to use FErc20DecimalsDelegate

change to use FErc721IsApprovedForAllDelegate, FErc1155IsApprovedForAllDelegate

change to use FGetNFTSupplyDelegate

change to use Erc1155BalanceOfBatch

remove FGeneralCronosCallDelegate

use int64_t instead of int64
leejw51crypto added a commit that referenced this issue Mar 28, 2023
Solution: convert sync apis to async

change GetEthBalance to async

change GetBalance to async

convert GetNFTSupply to async

convert GetNFTOwner to async

convert GetNFTCollection to async

convert GetNFTDenom to async

change GetNFTDenomByName to async

convert GetNFTAllDenoms to async

refactor sendamount

make SendAmount to async

convert GetNFTToken to async

convert Erc20Balance to async

convert Erc721Balance to async

convert Erc1155Balance to async

convert Erc1155BalanceOfBatch to async

convert Erc721Name to async

convert Erc721Symbol to async

convert Erc721Uri to async

convert Erc721GetApproved to async

convert Erc721IsApprovedForAll to async

convert Erc1155Uri to async

convert Erc1155IsApprovedForAll to async

convert Erc721Owner to async

convert Erc721TotalSupply to async

convert Erc721TokenByIndex to async

convert Erc721TokenOwnerByIndex to async

convert Erc20Name to async

make Erc20Symbol to async

make Erc20Decimals to async

make Erc20TotalSupply to async

make Erc20Allowance to async

tidy up comment

fix uint64 overflow

fix compile

remove legacy

change to use FErc20DecimalsDelegate

change to use FErc721IsApprovedForAllDelegate, FErc1155IsApprovedForAllDelegate

change to use FGetNFTSupplyDelegate

change to use Erc1155BalanceOfBatch

remove FGeneralCronosCallDelegate

use int64_t instead of int64
leejw51crypto added a commit that referenced this issue Mar 29, 2023
Solution: convert sync apis to async

change GetEthBalance to async

change GetBalance to async

convert GetNFTSupply to async

convert GetNFTOwner to async

convert GetNFTCollection to async

convert GetNFTDenom to async

change GetNFTDenomByName to async

convert GetNFTAllDenoms to async

refactor sendamount

make SendAmount to async

convert GetNFTToken to async

convert Erc20Balance to async

convert Erc721Balance to async

convert Erc1155Balance to async

convert Erc1155BalanceOfBatch to async

convert Erc721Name to async

convert Erc721Symbol to async

convert Erc721Uri to async

convert Erc721GetApproved to async

convert Erc721IsApprovedForAll to async

convert Erc1155Uri to async

convert Erc1155IsApprovedForAll to async

convert Erc721Owner to async

convert Erc721TotalSupply to async

convert Erc721TokenByIndex to async

convert Erc721TokenOwnerByIndex to async

convert Erc20Name to async

make Erc20Symbol to async

make Erc20Decimals to async

make Erc20TotalSupply to async

make Erc20Allowance to async

tidy up comment

fix uint64 overflow

fix compile

remove legacy

change to use FErc20DecimalsDelegate

change to use FErc721IsApprovedForAllDelegate, FErc1155IsApprovedForAllDelegate

change to use FGetNFTSupplyDelegate

change to use Erc1155BalanceOfBatch

remove FGeneralCronosCallDelegate

use int64_t instead of int64

add changelog
leejw51crypto added a commit to leejw51crypto/play-unreal-plugin that referenced this issue Mar 29, 2023
Solution: convert sync apis to async

change GetEthBalance to async

change GetBalance to async

convert GetNFTSupply to async

convert GetNFTOwner to async

convert GetNFTCollection to async

convert GetNFTDenom to async

change GetNFTDenomByName to async

convert GetNFTAllDenoms to async

refactor sendamount

make SendAmount to async

convert GetNFTToken to async

convert Erc20Balance to async

convert Erc721Balance to async

convert Erc1155Balance to async

convert Erc1155BalanceOfBatch to async

convert Erc721Name to async

convert Erc721Symbol to async

convert Erc721Uri to async

convert Erc721GetApproved to async

convert Erc721IsApprovedForAll to async

convert Erc1155Uri to async

convert Erc1155IsApprovedForAll to async

convert Erc721Owner to async

convert Erc721TotalSupply to async

convert Erc721TokenByIndex to async

convert Erc721TokenOwnerByIndex to async

convert Erc20Name to async

make Erc20Symbol to async

make Erc20Decimals to async

make Erc20TotalSupply to async

make Erc20Allowance to async

tidy up comment

fix uint64 overflow

fix compile

remove legacy

change to use FErc20DecimalsDelegate

change to use FErc721IsApprovedForAllDelegate, FErc1155IsApprovedForAllDelegate

change to use FGetNFTSupplyDelegate

change to use Erc1155BalanceOfBatch

remove FGeneralCronosCallDelegate

use int64_t instead of int64

add changelog
leejw51crypto added a commit that referenced this issue Mar 30, 2023
Solution: convert sync apis to async

change GetEthBalance to async

change GetBalance to async

convert GetNFTSupply to async

convert GetNFTOwner to async

convert GetNFTCollection to async

convert GetNFTDenom to async

change GetNFTDenomByName to async

convert GetNFTAllDenoms to async

refactor sendamount

make SendAmount to async

convert GetNFTToken to async

convert Erc20Balance to async

convert Erc721Balance to async

convert Erc1155Balance to async

convert Erc1155BalanceOfBatch to async

convert Erc721Name to async

convert Erc721Symbol to async

convert Erc721Uri to async

convert Erc721GetApproved to async

convert Erc721IsApprovedForAll to async

convert Erc1155Uri to async

convert Erc1155IsApprovedForAll to async

convert Erc721Owner to async

convert Erc721TotalSupply to async

convert Erc721TokenByIndex to async

convert Erc721TokenOwnerByIndex to async

convert Erc20Name to async

make Erc20Symbol to async

make Erc20Decimals to async

make Erc20TotalSupply to async

make Erc20Allowance to async

tidy up comment

fix uint64 overflow

fix compile

remove legacy

change to use FErc20DecimalsDelegate

change to use FErc721IsApprovedForAllDelegate, FErc1155IsApprovedForAllDelegate

change to use FGetNFTSupplyDelegate

change to use Erc1155BalanceOfBatch

remove FGeneralCronosCallDelegate

use int64_t instead of int64

add changelog

use int64 instead of int64_t

refactor try,catch and AsyncTask
leejw51crypto added a commit to leejw51crypto/play-unreal-plugin that referenced this issue Mar 30, 2023
Solution: convert sync apis to async

change GetEthBalance to async

change GetBalance to async

convert GetNFTSupply to async

convert GetNFTOwner to async

convert GetNFTCollection to async

convert GetNFTDenom to async

change GetNFTDenomByName to async

convert GetNFTAllDenoms to async

refactor sendamount

make SendAmount to async

convert GetNFTToken to async

convert Erc20Balance to async

convert Erc721Balance to async

convert Erc1155Balance to async

convert Erc1155BalanceOfBatch to async

convert Erc721Name to async

convert Erc721Symbol to async

convert Erc721Uri to async

convert Erc721GetApproved to async

convert Erc721IsApprovedForAll to async

convert Erc1155Uri to async

convert Erc1155IsApprovedForAll to async

convert Erc721Owner to async

convert Erc721TotalSupply to async

convert Erc721TokenByIndex to async

convert Erc721TokenOwnerByIndex to async

convert Erc20Name to async

make Erc20Symbol to async

make Erc20Decimals to async

make Erc20TotalSupply to async

make Erc20Allowance to async

tidy up comment

fix uint64 overflow

fix compile

remove legacy

change to use FErc20DecimalsDelegate

change to use FErc721IsApprovedForAllDelegate, FErc1155IsApprovedForAllDelegate

change to use FGetNFTSupplyDelegate

change to use Erc1155BalanceOfBatch

remove FGeneralCronosCallDelegate

use int64_t instead of int64

add changelog

use int64 instead of int64_t

refactor try,catch and AsyncTask
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants