Skip to content

Karim camara/payment integration#73

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

Karim camara/payment integration#73
KarimJC wants to merge 14 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.)
Image

Image Cannot create multiple accounts for same org image image Org does not have a Stripe account image image Org does not have a Stripe account image

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

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think we would not want to name this test_key bc then we'll need to update the code when we switch out of test mode. better to keep it a generic name imo


func (sc *StripeClient) CreatePaymentIntent(ctx context.Context, input *models.CreatePaymentIntentInput) (*models.CreatePaymentIntentOutput, error) {

const applicationFeePercentage = 10 // CHANGE THIS TO BE THE APPLICATION FEE PERCENTAGE
Copy link
Collaborator

Choose a reason for hiding this comment

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

can be handled later - but i feel like this percent might differ by organization



ALTER TABLE event_occurrence
ADD COLUMN IF NOT EXISTS price INTEGER NOT NULL DEFAULT 0; -- Price in bhat
Copy link
Collaborator

Choose a reason for hiding this comment

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

can we add currency here as well

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

Comments