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

Adds Kadena example #70

Merged
merged 16 commits into from
Jul 14, 2023
Merged

Adds Kadena example #70

merged 16 commits into from
Jul 14, 2023

Conversation

ash-vd
Copy link
Contributor

@ash-vd ash-vd commented Oct 29, 2022

No description provided.

@vercel
Copy link

vercel bot commented Oct 29, 2022

@ashwinvandijk is attempting to deploy a commit to the WalletConnect Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

@Billiej88 Billiej88 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some changes

@ash-vd ash-vd force-pushed the feat/kadena-example branch 3 times, most recently from 14ac366 to 12f7a0a Compare November 22, 2022 20:44
@ash-vd ash-vd force-pushed the feat/kadena-example branch 3 times, most recently from 3544328 to 80827ba Compare November 23, 2022 11:31
@ash-vd ash-vd marked this pull request as ready for review November 23, 2022 12:16
Copy link

@Billiej88 Billiej88 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is nis

@ash-vd
Copy link
Contributor Author

ash-vd commented Nov 30, 2022

@bkrem This one is ready for review if you (or somebody of the WC-team) has the time. Thanks!

@Randynamic
Copy link

@chadyj We submitted an example of how to make a connection to the Kadena blockchain. Can you take a look at it so we can move on, on our side, if this is the way to go?

@bkrem
Copy link
Member

bkrem commented Dec 5, 2022

Hi @ashwinvandijk, @Randynamic,

Thanks for putting in time to integrate Kadena with our examples. On our side every additional chain also entails additional maintenance and complexity considerations for our team, so we're generally careful when it comes to adding new chains unless it's a no-brainer.

We're not familiar with Kadena (which may well be on us of course!), but since you seem to be your own L1, could you highlight some dapps and wallets that already support Kadena?

Generally we recommend teams for less widely known chains to fork the examples, add support (as you have here) and thus use our examples to "piggy-back" example apps/wallets for ecosystem-specific usage :)

@Randynamic
Copy link

Randynamic commented Jan 9, 2023

Hey @bkrem,

Sorry totally missed your reply. Best wishes for 2023!
It's hard to hear you haven't heard about us yet 😉. I was hoping we were already in the circle of well-known chains out there, but I guess we still need to widen our reach hehe.
We have several wallets that support Kadena:

Here you can find a list of projects on our platform. And we would love them all to work with Wallet Connect.

Maintenance-wise, our team/community can help on that part if we need to make changes, etc.

Otherwise, we will indeed add it to our own repo. But it would be great to get this PR merged.

Hope to hear from you soon.

Cheers!

@floordebest
Copy link

floordebest commented Jan 9, 2023

Developer of Linx Wallet here, i support this PR to the fullest. Would be great if we could get some documentation about the Kadena standard in the WalletConnect Docs / example repo.

We are about to launch our wallet in beta with full WalletConnect v2 support for Kadena, having this information easily available for dApp developers is pretty important.

Lots of projects are currently building on Kadena as it is a scalable POW network.

@tim-eucalyptus
Copy link

I endorse this too, please merge. The PR itself looks cleanly written. Just need to fix the merge conflicts, @ash-vd .

@bkrem
Copy link
Member

bkrem commented Jun 20, 2023

@ash-vd Apologies that this PR stalled out.

Could you update the PR + fix conflicts, then we'll get this reviewed and merged for good 🙏

@vercel
Copy link

vercel bot commented Jul 7, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
react-dapp-v2 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 14, 2023 10:27am
react-wallet-v2 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 14, 2023 10:27am

@bkrem
Copy link
Member

bkrem commented Jul 7, 2023

@ash-vd Dapp build is failing due to an outdated dependency import. I've added two tweaks that should fix the build.

@ash-vd
Copy link
Contributor Author

ash-vd commented Jul 13, 2023

@bkrem Just a subtle nudge that this is ready for another try :)

Copy link
Member

@bkrem bkrem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for getting us back into shape here @ash-vd 🙏

Some minor comments but no blockers. Will merge as soon as these are resolved 🚀

wallets/react-wallet-v2/package.json Outdated Show resolved Hide resolved
wallets/react-wallet-v2/package.json Outdated Show resolved Hide resolved
dapps/react-dapp-v2/src/components/Asset.tsx Outdated Show resolved Hide resolved
dapps/react-dapp-v2/src/chains/kadena.ts Outdated Show resolved Hide resolved
@bkrem
Copy link
Member

bkrem commented Jul 14, 2023

@ash-vd awesome thanks for the quick fixes, all good from my side 🚀

Think it just needs a final sync with upstream main so that I can merge, probably lockfile conflicts:
Screenshot 2023-07-14 at 11 48 05

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

Successfully merging this pull request may close these issues.

6 participants