Skip to content

Karim camara/payment integration#73

Draft
KarimJC wants to merge 8 commits intomainfrom
Karim-Camara/Payment-Integration
Draft

Karim camara/payment integration#73
KarimJC wants to merge 8 commits intomainfrom
Karim-Camara/Payment-Integration

Conversation

@KarimJC
Copy link
Contributor

@KarimJC KarimJC commented Feb 11, 2026

This is what I got so far - I just need to finish testing, implement payment intents in registrations and create webhooks.

Description

[Link to Ticket](insert the link to your ticket inside the parenthesis here)
Please include a summary of the changes and the related issue. Please also
include relevant motivation, context, and images!

How Has This Been Tested?

Please describe the tests that you manually ran to verify your changes (beyond any unit/integration tests written and ran).

Screenshots

Please provide screenshots of manual testing (Scalar, frontend pages, etc.)

Checklist

General

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas

Backend Changes (if applicable)

  • I have written unit tests for my code
  • I have written integration tests where appropriate
  • New and existing unit tests pass locally with my changes
  • I have run make api-validate and make api-gen to keep the openAPI spec up to date
  • I have tested error handling and edge cases
  • Database migrations are included (if applicable)

Frontend Changes (if applicable)

  • I have tested in multiple browsers (Chrome, Firefox, Safari)
  • Mobile responsive design verified (if applicable)
  • I have run make generate-api to keep the API hooks up to date
  • For mobile features: Tested on both iOS and Android devices
  • For mobile features: Screenshots from both iOS and Android included below
  • No console errors or warnings
  • Loading states and error states handled

Mobile Screenshots (if applicable)

iOS

[Add iOS screenshots here]

Android

[Add Android screenshots here]

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove these temp files pls!

supabase/.temp/cli-latest
supabase/.branches/_current_branch

StripePaymentMethodID string `json:"stripe_payment_method_id" db:"stripe_payment_method_id"`
CardBrand *string `json:"card_brand,omitempty" db:"card_brand"`
CardLast4 *string `json:"card_last4,omitempty" db:"card_last4"`
CardExpMonth *int `json:"card_exp_month,omitempty" db:"card_exp_month"`
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we need to store any of this card info or can it be pulled from stripe?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feat: Implement Registration & Payment Routes with Stripe Integration Continuation Feat: Create a Cleint to Enable Payements Through Stripe

2 participants