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

Fix anonymous zaps #938

Merged
merged 1 commit into from
Mar 25, 2024
Merged

Fix anonymous zaps #938

merged 1 commit into from
Mar 25, 2024

Conversation

benthecarman
Copy link
Collaborator

MutinyWallet/mutiny-node#1040 broke anon zaps, now requires you to put the type of zap you're doing.

We should add public and private zaps eventually, but this fixes the functionality for now

@benthecarman benthecarman added the bug Something isn't working label Mar 25, 2024
@benthecarman benthecarman requested a review from futurepaul March 25, 2024 20:57
Copy link

cloudflare-workers-and-pages bot commented Mar 25, 2024

Deploying mutiny-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9b9a273
Status: ✅  Deploy successful!
Preview URL: https://736864d0.mutiny-web.pages.dev
Branch Preview URL: https://fix-anon-zaps.mutiny-web.pages.dev

View logs

Copy link
Collaborator

@futurepaul futurepaul left a comment

Choose a reason for hiding this comment

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

I don't have a good way to test but I trust you

@benthecarman
Copy link
Collaborator Author

I don't have a good way to test

you can zap the faucet

@benthecarman benthecarman merged commit c9ebb64 into master Mar 25, 2024
5 checks passed
@benthecarman benthecarman deleted the fix-anon-zaps branch March 25, 2024 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants