keep-starknet-strange / joyboy

Decentralized social built with Nostr and powered by Starknet account abstraction.
https://joyboy.community
MIT License
68 stars 50 forks source link

Fix Transaction Failure Issue When Tipping Using ArgentX Desktop #245

Closed lauchaves closed 1 month ago

lauchaves commented 1 month ago

Description

Issue

Expected

Screenshots

image

image

ugur-eren commented 1 month ago

Hi clicking the copy button next to the error message will copy the whole error message. Can you share that?

resim

lauchaves commented 1 month ago

hey! @ugur-eren here it is

Error in the called contract (0x016d6b7d8ba15d1903ea942a6e9414a70b0f1b2219215dc46faca9a38a522d06): Error at pc=0:15213: Got an exception while executing a hint: Requested contract address ContractAddress(PatriciaKey(StarkFelt("0x078a022e6906c83e049a30f7464b939b831ecbe47029480d7e89684f20c8d263"))) is not deployed. Cairo traceback (most recent call last): Unknown location (pc=0:222) Unknown location (pc=0:4845) Unknown location (pc=0:10647) Unknown location (pc=0:15231)

ugur-eren commented 1 month ago

Are you trying on the mainnet? Currently only sepolia is supported.

ugur-eren commented 1 month ago

This issue can be assigned to contributors who want to implement it.s

There should be a check for connect account's chain id. If the chain is not Starknet Sepolia, app should show a uncloseable dialog with title: 'Wrong Network' message: 'Joyboy currently only supports Starknet Sepolia network. Please switch to Sepolia network to continue.'

lauchaves commented 1 month ago

got it!, yeah its mainnet image

lauchaves commented 1 month ago

This issue can be assigned to contributors who want to implement it.s

There should be a check for connect account's chain id. If the chain is not Starknet Sepolia, app should show a uncloseable dialog with title: 'Wrong Network' message: 'Joyboy currently only supports Starknet Sepolia network. Please switch to Sepolia network to continue.'

@ugur-eren If you agree, I can take care of this 😃

lauchaves commented 1 month ago

JM @ugur-eren this is ready for review! Here's the PR