starknet-react icon indicating copy to clipboard operation
starknet-react copied to clipboard

Release V3 alpha/beta

Open rsodre opened this issue 1 year ago • 19 comments

I'm at a point where I can't stay on starknet v5 anymore, I have to upgrade to v6. But starknet-react is stuck on v5, hooks do not work if my project resolves starknet v6.

Please publish the @starknet-react/core next-v3 branch as alpha or beta!

rsodre avatar Jun 07 '24 03:06 rsodre

Hello, yes we finished migrating all (most) hooks to the new version and are going to release the next one. Curious to hear why you cannot use starknet v5 anymore? AFAIK it should not break until all the ecosystem migrated to starknet v6.

fracek avatar Jun 07 '24 09:06 fracek

I'm working with Dojo, and dojo.js already migrated to v6, preparing to add Paymaster support on execute calls. With v5, I start getting errors, because they already use some new types. With v6, some of your core hooks stop working.

I'm stuck, about to recreate some of your hooks just so I can continue working. That's why I'm asking to release an alpha.

Or... is there another way to add the v3 branch as a dependency? As a monorepo, I can't just use the github branch link.

rsodre avatar Jun 07 '24 12:06 rsodre

Do you need to support wallets like braavos and argent or you have a smart wallet? I'm happy to push a pre-release this week but in that case you'll need to implent your own connector to work with browser wallets (looks like braavos already works with v6 tho).

fracek avatar Jun 10 '24 10:06 fracek

Not at this moment, but I will in a week or two.

Looks like the only hook I'm using that is having issues on v6 is useBalance(). I was able to remake it using useContractRead() (referenced above) and my urgency has lowered.

rsodre avatar Jun 11 '24 14:06 rsodre

That sounds good. I'll start making pre-releases of v3 but wallet support is still not 100% complete. Worst case someone will have to implement an adapter for the v6 to v5 wallet api.

fracek avatar Jun 12 '24 10:06 fracek

jumping into this discussion, i have also faced issues with Argent when upgrading v5 -> v6. So far braavos is working well.

jrcarlos2000 avatar Jun 18 '24 13:06 jrcarlos2000

I started testing wallets this week, useBlockNumber()` also has some issues, and I get all sort of weird errors.

A pre-release would be good!

rsodre avatar Jun 19 '24 01:06 rsodre

Hello, I started publishing pre-releases (3.0.0-beta.0 is the current one). I'm testing it with the most recent versions of braavos and argent.

fracek avatar Jun 23 '24 09:06 fracek

Hi @fracek is there any docs we can start looking?

jhintux avatar Jul 02 '24 16:07 jhintux

The preview is available here. Will add instructions on how to update shortly.

fracek avatar Jul 02 '24 20:07 fracek

Hi Team, any ETA of the stable version? Starknet-react is the core of our project. You are an amazing team, thank you so much for all your effort!

gianalarcon avatar Jul 19 '24 11:07 gianalarcon

@gianalarcon Sorry for the late reply, I missed this comment! You can already upgrade by switching to the @next tag. Everything is implemented and only small bugs need to be ironed out. Waiting for the full ecosystem to update before making a final release.

fracek avatar Jul 24 '24 09:07 fracek

gm Team, wondering when will the @next version have a stable released ? @fracek 👀

jrcarlos2000 avatar Aug 23 '24 15:08 jrcarlos2000

There's still a couple of small items related to in-app wallets before we tag the release as latest. Want to avoid breaking changes after it's an official release.

fracek avatar Aug 27 '24 19:08 fracek

Hi @fracek , I was wondering whether the version on the main branch is the intended stable version? if not do you have an estimate time of the release? I've noticed significant performance drops on the v3 account connections compared to its predecessors.

metalboyrick avatar Sep 19 '24 06:09 metalboyrick

Hi @fracek , I was wondering whether the version on the main branch is the intended stable version? if not do you have an estimate time of the release?

Yes v3 is the main release now.

I've noticed significant performance drops on the v3 account connections compared to its predecessors.

Can you provide more details? Where are you experiencing slower performance? We also have a demo to measure latency of some wallet operations, it would help if you can click on all buttons and share what latency you get.

fracek avatar Sep 19 '24 07:09 fracek

@fracek thanks for the demo! very helpful!

As for how i concluded the perf issue, I noticed that when trying to connect, it takes a while for update the connected state of the wallet (from useAccount, disconnected to connected). I've also noticed that argentX is worse in the duration of connection. This did not happen in the previous version. Do you have any indication of potential issues and how to fix?

Slow argentX: image

metalboyrick avatar Sep 19 '24 09:09 metalboyrick

The issue seems to be related to the new wallet.request API. I will double check if there's also any issue on our side, but if the issue is slow response from wallets there's not much we can do.

fracek avatar Sep 19 '24 09:09 fracek

@fracek Thanks for the hard work!

metalboyrick avatar Sep 19 '24 13:09 metalboyrick