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

Added Crypto Donation Option with WalletConnect Integration #32

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ayan-joshi
Copy link

Closes Issue #22
I added the wallet connect integration button to donation form now people can scan qr and donate via crypto

image

have a look @phyleria

Added wallet connect to integrate crypto
@ayan-joshi
Copy link
Author

@todak2000 thanks !

@ayan-joshi
Copy link
Author

@phyleria can u merge this one

Copy link
Contributor

@uchibeke uchibeke left a comment

Choose a reason for hiding this comment

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

Where are you getting donation type wallet-connect from? This is not an existing donation type. See the src/types/non-profit.d.ts. So should add a new crypto type and define how it should be structured in the data: consider wallet address, chain, accepted tokens (or accept any token), etc

Also, a Wallet Connect integration, as you've proposed, should complete the donation flow and the actual donation should be sent to a wallet address of a non profit in the non profit donation data.

  • Select Non Profit
  • Select donate via Crypto
  • Enter Amount to Donate
  • Connect Wallet
  • Send a Token to the address

You should also consider:

  • Chain the Non profit accepts

Finally, please ensure all used packages are added to package.json. You use @walletconnect/web3-provider but its not installed.
Screenshot 2024-10-19 at 9 39 08 AM

Let me know if you need help with this or if you need a more detailed implementation step to get this done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants