Skip to content
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

[Stripe] Implement INSERT, UPDATE, and DELETE for the Payment Intents table #7874

Closed
martyna-mindsdb opened this issue Oct 18, 2023 · 9 comments · Fixed by #8094
Closed

[Stripe] Implement INSERT, UPDATE, and DELETE for the Payment Intents table #7874

martyna-mindsdb opened this issue Oct 18, 2023 · 9 comments · Fixed by #8094
Assignees
Labels
first-timers-only Good issues for the users that contribute for the first time. hacktoberfest Contirubte to MindsDB and participate in Hacktoberfest. help wanted Extra attention is needed

Comments

@martyna-mindsdb
Copy link
Collaborator

martyna-mindsdb commented Oct 18, 2023

MindsDB integrates with Stripe, so you can fetch data from Stripe and use it to make relevant predictions and forecasts with MindsDB.

Resources

Familiarize yourself with the existing Stripe integration: https://github.com/mindsdb/mindsdb/tree/staging/mindsdb/integrations/handlers/stripe_handler

Steps to Follow

The task is to extend this implementation to include the INSERT, UPDATE, and DELETE commands for the Payment Intents table.

📢 Additional rewards 🏆

Each integration extension brings 15+ points for SWAG and entry into the draw for a 👇 :

ℹ️ For more info check out https://mindsdb.com/hacktoberfest/ 👈

Please note that this issue is for first-time contributors to MindsDB

@martyna-mindsdb martyna-mindsdb added help wanted Extra attention is needed hacktoberfest Contirubte to MindsDB and participate in Hacktoberfest. first-timers-only Good issues for the users that contribute for the first time. labels Oct 18, 2023
@Sekhar-Kumar-Dash
Copy link
Contributor

@martyna-mindsdb can you please assign this issue to me

@AaryaNale
Copy link

@martyna-mindsdb Can I work on this issue?

@martyna-mindsdb
Copy link
Collaborator Author

@AaryaNale
Note that this issue has been already assigned. You can look at other open issues.

@Sekhar-Kumar-Dash
Copy link
Contributor

@martyna-mindsdb please review the pull request

@Sekhar-Kumar-Dash
Copy link
Contributor

After some testing i found out that stripe does not support delete payments yet but we can modify the delete method so that it will cancel the payment and update the status at canceled_at column .

@martyna-mindsdb
Copy link
Collaborator Author

@Sekhar-Kumar-Dash
If DELETE is not supported for payments, then please ignore it - implement only INSERT and UPDATE.

(The payment can be canceled using UPDATE and setting the canceled_at column value.)

@Sekhar-Kumar-Dash
Copy link
Contributor

@Sekhar-Kumar-Dash If DELETE is not supported for payments, then please ignore it - implement only INSERT and UPDATE.

(The payment can be canceled using UPDATE and setting the canceled_at column value.)

okay

@Sekhar-Kumar-Dash
Copy link
Contributor

Sekhar-Kumar-Dash commented Oct 28, 2023

@Sekhar-Kumar-Dash If DELETE is not supported for payments, then please ignore it - implement only INSERT and UPDATE.
(The payment can be canceled using UPDATE and setting the canceled_at column value.)
@martyna-mindsdb it would be better if we cancel the payment with delete method instead of update

@martyna-mindsdb
Copy link
Collaborator Author

@MinuraPunchihewa ^

@chandrevdw31 chandrevdw31 moved this from In progress to Done in Community Contributors Nov 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
first-timers-only Good issues for the users that contribute for the first time. hacktoberfest Contirubte to MindsDB and participate in Hacktoberfest. help wanted Extra attention is needed
3 participants