gatsbyjs / gatsby

The best React-based framework with performance, scalability and security built in.
https://www.gatsbyjs.com
MIT License
55.24k stars 10.32k forks source link

Square Payment Processing missing second crucial half of instructions #26055

Closed amandakievet closed 3 years ago

amandakievet commented 4 years ago

The documentation about how to use Square with Gatsby here goes through how to get a nonce but not how to actually capture a payment. Looking through the Square documentation, it seems like you need to have a server side function to actually capture a payment. Can someone help me understand how that's possible with Gatsby as a static site?

Thank you

jonniebigodes commented 4 years ago

@amandakievet i picked on your issue as i was the contributor that created the documentation based on @AishaBlake original issue. As a complement to that documentation i have a example that is mentioned in the documentation. But also there's a starter that i made. You can take a look at it here.

In said starter i used a netlify function to handle some of the integration with Square SDK.

Take a look at the functions folder and you'll see what i mean.

Also as a side note, i know that some of dependencies might be a bit outdated. But honestly i hadn't had the time to take care of the repo and update what needs to be updated. But the repo should at least get you a perspective on what you intend to do.

Stay safe

github-actions[bot] commented 4 years ago

Hiya!

This issue has gone quiet. Spooky quiet. πŸ‘»

We get a lot of issues, so we currently close issues after 30 days of inactivity. It’s been at least 20 days since the last update here. If we missed this issue or if you want to keep it open, please reply here. You can also add the label "not stale" to keep this issue open! As a friendly reminder: the best way to see this issue, or any other, fixed is to open a Pull Request. Check out gatsby.dev/contribute for more information about opening PRs, triaging issues, and contributing!

Thanks for being a part of the Gatsby community! πŸ’ͺπŸ’œ

meganesu commented 4 years ago

Adding some extra notes to summarize so that hopefully someone can pick this issue up during Hacktoberfest!

Steps to resolve this issue

Open a pull request

imshubhamsingh commented 4 years ago

Hi @meganesu. I just recently started learning Gatsby. This would be a great start for me as well. I would love to pick this up. I'll share PR by the end of this weekend. Thanks ✌️

jonniebigodes commented 4 years ago

@meganesu the starter's dependencies and example will be updated towards the end of the week. I've been trying but i haven't got around to get some time and update it. Once i've applied the changes i'll let you know.

Stay safe

meganesu commented 4 years ago

@imshubhamsingh @jonniebigodes sounds good to both! Feel free to reach out if you have any questions.

imshubhamsingh commented 3 years ago

@meganesu I've updated the doc and example project. Do let me know if I missed something. This is my first ever PR in gatsby. Thanks ✌️

amandakievet commented 3 years ago

Thanks all!

davidhowland commented 3 years ago

@imshubhamsingh thank you for adding the missing documentation - it's extremely helpful. As I was reading through I noticed that your example Netlify lambda function uses the deprecated square-connect sdk instead of the new node.js SDK released in December. Is there a specific reason for this that I missed somewhere?

imshubhamsingh commented 3 years ago

Hey @davidhowland, thanks for informing me about it. As PR #27272 was opened long back, it was written keeping in mind which ever version was latest at that point of time. At the moment I'm occupied with other work. If this is something you want to pickup, you can. Otherwise I can send in a separate PR with updated examples and doc using latest version in coming weeks.

davidhowland commented 3 years ago

@imshubhamsingh ah, that makes sense. I missed the original date here. Thanks! I am currently trying to figure out how to implement this payment form via Netlify functions and have not been successful so any help that you have the time to provide here would be greatly appreciated.

LekoArts commented 3 years ago

The original issue was addressed. If you want to keep the docs up-to-date feel free to contribute a PR. Thanks!