search

anza-xyz / solana-pay

A new standard for decentralized payments.
https://solanapay.com
Apache License 2.0
1.29k stars 450 forks source link

Check for amount in query params #161

Closed Joshua-Ogbonna closed 1 year ago

Joshua-Ogbonna commented 1 year ago

Hi, we were using the Solana-pay point-of-sale for a project at our workplace. We didn't want users to use the calculator as the amount would be coming from our product. This will enable users to pay the exact amount that is coming from the website. We were wondering if you could merge this PR.

Basically, I added a condition in the pending page to check for amount params in the URL.

vercel[bot] commented 1 year ago

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

Name Status Preview Updated
solana-pay ✅ Ready (Inspect) Visit Preview Sep 22, 2022 at 6:15PM (UTC)
solana-pay-docs ✅ Ready (Inspect) Visit Preview Sep 22, 2022 at 6:15PM (UTC)
jordaaash commented 1 year ago

The point of sale app is only a reference implementation. It's not intended to accommodate for new use cases. Many of the changes in this PR are specific to your application, business logic, even your website URLs. This should probably be a PR against your own repo :)

Joshua-Ogbonna commented 1 year ago

Okay. Thank you very much.

The application has helped me a lot. I later found an error with my implementation.

On Thu, 22 Sept 2022 at 19:24, Jordan Sexton @.***> wrote:

The point of sale app is only a reference implementation. It's not intended to accommodate for new use cases. Many of the changes in this PR are specific to your application, business logic, even your website URLs. This should probably be a PR against your own repo :)

— Reply to this email directly, view it on GitHub https://github.com/solana-labs/solana-pay/pull/161#issuecomment-1255392361, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOAJHFSUKXCQ3GVGDZ4KHC3V7SP4RANCNFSM6AAAAAAQSZGXCI . You are receiving this because you authored the thread.Message ID: @.***>