-
Notifications
You must be signed in to change notification settings - Fork 45
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
WIP: Add signup.cash to js dev tools #42
base: master
Are you sure you want to change the base?
Conversation
Deploy preview for bch ready! Built with commit 892cf92 |
The code in the PR looks good, but I haven't tried the package. It doesn't say what the package is for in the docs. EDIT: it does explain in the following sections :) @imaginaryusername what do you think about adding this tool? |
I just tested wallet.signup.cash and it is not sending transactions. It seems to be powered by this tool, so I would recommend waiting for a more stable release before listing it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sending transactions is broken on https://wallet.signup.cash
@merc1er May you try the example app for payment and let me know if it works? Thanks https://signup-examples-v1.vercel.app/example_simple_tipping_button.html |
What is the expected behaviour? It seems to be working on my end: |
It seems to be working now, but once in a while i get this: Actually it redirects me to this URL: https://wallet.signup.cash/send which returns a 404 |
@merc1er I think you were reloading the page on different paths. But good catch, forgot to redirect 404 to index. Already fixed. |
A new user left a comment. This user must be approved by a Netlify team owner before comments can be displayed. |
No description provided.