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

Enable Web3Modal on Polygon Checkout #10530

Closed
nfrgosselin opened this issue Apr 25, 2022 · 3 comments · Fixed by #10564
Closed

Enable Web3Modal on Polygon Checkout #10530

nfrgosselin opened this issue Apr 25, 2022 · 3 comments · Fixed by #10564
Assignees
Labels

Comments

@nfrgosselin
Copy link
Collaborator

nfrgosselin commented Apr 25, 2022

User Story:
As a grant donor
I want to checkout on Polygon with wallets other than Metamask
So that I am not tied to using one wallet

Acceptance Criteria
GIVEN that I am on /grants/cart AND have Polygon-eligible grants in my cart AND do not have a wallet connected
WHEN I click "Polygon checkout"
THEN I see the Web3Modal to select a wallet

GIVEN that I am on /grants/cart AND have Polygon-eligible grants in my cart AND have a Web3Modal-eligible wallet connected
WHEN I click "Polygon checkout"
THEN I am able to checkout

Product & Design Links:

Tech Details:
@chibie has documented the required steps for this here

Open Questions:

Notes/Assumptions:
Follow-on story from #10482

@nfrgosselin
Copy link
Collaborator Author

@chibie I was able to test Coinbase end-to-end and the integration worked successfully.

However, when I tried to use Portis,Fortmatic, and Wallet connect I could go through the login flow fine, then I would get this error:
image

Any ideas?

@chibie
Copy link
Contributor

chibie commented May 8, 2022

@nfrgosselin Coinbase changes aren't in this PR but WIP here -> #10410. did you mean MetaMask?

for the other wallet types, were you on mainnet/rinkeby?

@chibie
Copy link
Contributor

chibie commented May 10, 2022

this pr fixes the bug -> #10606

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

Successfully merging a pull request may close this issue.

2 participants