-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: merchant integration #1
Comments
Really happy to see some feedback! Nice idea adding first class stripe support, can you elaborate on that? I was thinking about splitting the code up into a monorepo to extract the helper logic and additional packages eg stripe support into their own package. |
I'm understanding the intent here a little better now. Issue:
If any of all of this is out of scope, please feel free to be direct. :) This may just be one "sub-module" structurally, as even a SaaS Subscription Payments Starter
Are there more things I missed? Franky, I'm not crystal clear on the latest Very happy this is something you would entertain doing, as a first-class solution will save some long nights and lost hairs for the community... P.S. Styling?I closed the styling feature as not worthwhile. If you are considering putting in cycles stubbing out a secure Subscription CRUD implementation, that makes pursing #2 more desirable, just re-open it and assign it to me and I'll send PR's adding in tailwind... Best, |
Thanks for claryfying. I agree, we definately need a starter repo! And I´d add adyen to the merchant list as i´m using it. One question though, do you want to just create a starter project or also publish packages like |
I'm open to either. I'm happy to contribute and help pull it together, I'm specifically lacking the cycles spent in the bowels of the current progress on the
What are your theoughts? What I could contribute code-wise to make it easier assuming you're taking the lead on the JWT auth schema/logic? Happy to chat on discord or something like that to iterate quickly/easily... |
Hey @david-plugge
In response to you asking for feedback, here's a first swing...
Feature Description
.ts
so that they are easy to changeI would personally suggest
stripe
first, just due to being omnipresent...Questions
Is this in scope for what you are putting together?
Happy to help... you seem to have a strong handle on the new SK changes, JWT best practices and ts...
Thoughts?
The text was updated successfully, but these errors were encountered: