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

Release V3 alpha/beta #451

Closed
rsodre opened this issue Jun 7, 2024 · 19 comments
Closed

Release V3 alpha/beta #451

rsodre opened this issue Jun 7, 2024 · 19 comments

Comments

@rsodre
Copy link

rsodre commented Jun 7, 2024

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!

@fracek
Copy link
Contributor

fracek commented Jun 7, 2024

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.

@rsodre
Copy link
Author

rsodre commented Jun 7, 2024

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 added a commit to underware-gg/pistols that referenced this issue Jun 7, 2024
@fracek
Copy link
Contributor

fracek commented Jun 10, 2024

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).

@rsodre
Copy link
Author

rsodre commented Jun 11, 2024

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.

@fracek
Copy link
Contributor

fracek commented Jun 12, 2024

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.

@jrcarlos2000
Copy link

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

@rsodre
Copy link
Author

rsodre commented Jun 19, 2024

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!

@fracek
Copy link
Contributor

fracek commented Jun 23, 2024

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.

@edulanasca
Copy link

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

@fracek
Copy link
Contributor

fracek commented Jul 2, 2024

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

@gianalarcon
Copy link

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!

@fracek
Copy link
Contributor

fracek commented Jul 24, 2024

@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.

@jrcarlos2000
Copy link

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

@fracek
Copy link
Contributor

fracek commented Aug 27, 2024

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.

@metalboyrick
Copy link

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.

@fracek
Copy link
Contributor

fracek commented Sep 19, 2024

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.

@metalboyrick
Copy link

@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

@fracek
Copy link
Contributor

fracek commented Sep 19, 2024

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.

@metalboyrick
Copy link

@fracek Thanks for the hard work!

@fracek fracek closed this as completed Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants