Skip to content

feature: stripe implementation#43

Open
deb-coder-man wants to merge 2 commits intomainfrom
feature/stripe
Open

feature: stripe implementation#43
deb-coder-man wants to merge 2 commits intomainfrom
feature/stripe

Conversation

@deb-coder-man
Copy link
Copy Markdown

@deb-coder-man deb-coder-man commented Apr 28, 2026

What does this PR do?

Implemented stripe for membership payments. Created a simple sign up form that redirects the user to stripe payment, and then redirects to a confirmation screen upon successful payment. Members also gets created in members collection.

Type of change

  • Feature
  • Bug fix
  • Chore / config
  • Hotfix

Checklist

  • My branch follows the naming convention (feature/, fix/, chore/, hotfix/)
  • My commit messages follow the conventional commits format
  • CI checks pass

…rs and send payment link to the frontend, created a basic sign up form and a success screen for testing purposes
joengy
joengy previously approved these changes Apr 28, 2026
Copy link
Copy Markdown
Collaborator

@joengy joengy left a comment

Choose a reason for hiding this comment

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

Looks good to me !!

Richman-Tan
Richman-Tan previously approved these changes Apr 28, 2026
Copy link
Copy Markdown

@Richman-Tan Richman-Tan left a comment

Choose a reason for hiding this comment

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

LGTM

oorjagandhi
oorjagandhi previously approved these changes Apr 28, 2026
Copy link
Copy Markdown
Collaborator

@oorjagandhi oorjagandhi left a comment

Choose a reason for hiding this comment

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

LGTM once linter issues fixed.

Some concerns with membership signup but will address in the future

@deb-coder-man deb-coder-man dismissed stale reviews from oorjagandhi, Richman-Tan, and joengy via eb79aad April 28, 2026 22:20
@oorjagandhi oorjagandhi changed the title feature: strip implementation feature: stripe implementation Apr 29, 2026
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.

4 participants