-
Notifications
You must be signed in to change notification settings - Fork 2
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
[PR] Adding concepts clarification and tutorial #2
Conversation
Even though it's still in progress, @nelsonic you can take a look at the "theory" part and check if there's anything big missing in it, before I start doing an example. |
Added Stripe integration. It should be fully working, with authentication and |
Sounds good. keep going. looking forward to reading your research findings. 👌 |
This should be done. I've added the project, used DETS for storage (no need for PostgreSQL here) and I got it running with 100% coverage. Fixed some README typos and changed the I also added CI (but it didn't execute in this PR). |
README.md
Outdated
[That's a great question](https://www.indiehackers.com/post/stripe-vs-paddle-89161b0d5c). | ||
|
||
[Which seems to asked](https://splitbee.io/blog/why-we-moved-from-stripe-to-paddle) | ||
[over](https://stackshare.io/stackups/paddle-vs-stripe), |
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.
Please don't do this. [over] and [over] and [over]
is not good link text. dwyl/contributing#119
I understand that you are emphasising the fact that many people have had these same questions.
But as a document that we want to read, maintain and have others contribute to.
We want to set high standards for link text.
Please/thanks! 🙏
** (Mix) Expected one or more files/patterns to be given to mix format or for a .formatter.exs file to exist with an :inputs or :subdirectories key https://github.com/dwyl/learn-payment-processing/actions/runs/3828017445/jobs/6513143646 |
Codecov Report
@@ Coverage Diff @@
## main #2 +/- ##
========================================
Coverage ? 100.00%
========================================
Files ? 9
Lines ? 38
Branches ? 0
========================================
Hits ? 38
Misses ? 0
Partials ? 0 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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.
@LuchoTurtle this write-up is superb. 😍 Well done. 🎉
Note: I made a few tweaks but your core content is excellent. Keep up the great work. 🚀 |
closes #1
This PR should be the initial template for the project.
Has a section describing concepts for better understanding and a tutorial to follow through integration with Stripe.