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

Feat/stripe #226

Closed
wants to merge 371 commits into from
Closed

Feat/stripe #226

wants to merge 371 commits into from

Conversation

TeddyRoncin
Copy link
Member

@TeddyRoncin TeddyRoncin commented Sep 4, 2024

Changed payment method from Etupay to Stripe

Changes

  • Route POST /users/current/cart now returns the token to pay with Stripe.
  • Removed old callback routes for Etupay (GET/POST /etupay), and added POST /stripe/accepted and POST /stripe/expired routes to tell when a cart has been payed or expired.
  • Added route PATCH /admin/items/stripe-sync to synchronize items in Stripe.

Breaking changes

Everything related to payments, and cart status.

What's left to do

  • Verify build
  • Lint
  • Remove console.log / commented code
  • Test the expired route
  • Instantiate Stripe better

B4nanaJuice and others added 30 commits October 4, 2021 11:02
* feat: channel and role creation

* fix: channel and role creation

* fix: channel creation

* fix: put syncRoles function when the team locks

* fix: add clearRoles function for removing roles if the discord id changes

* fix: adding tests for the syncRoles and setupTeams functions and road to syncRoles

* fix: add tests and route

* fix: fixed route and added description for api documentation

* fix: added coaches and tournament resp to channels

* fix: last modifications

* fix: bot starting

* fix(tests): tests should have an end and should pass

* fix(tests): tests should actually end

* Fix: Discord (#75)

* feat: remake discord bot with only HTTP calls

* fix: change ts file to d.ts file

* fix: correct suggestions

* fix: correct color for team role

* fix: correct permissions

* fix: apply suggestions

* fix: increase coverage in some part of the code

* fix: lint

Co-authored-by: AlbanSdl <[email protected]>
Co-authored-by: Thomas de Lachaux <[email protected]>
* fix(misc): fix coach limit blocking new players in team

* smtp credentials are now in a an uri format
* empty tables will now longer be rendered in emails
* fixed localhost invalid redirect_uri for discord oauth

* docs: update discord.md and fake.md
* fix(tickets): optional cartItemId to return current user ticket

* tickets should not be sent as an email attachment when declared as sent in pdf format
* tickets should be base64-encoded

* test(tickets): fix coverage loss

* fix(env): forgot this line in previous pr
* fix(tickets): emails now don't send the tickets as attachments

* fix(tickets): fixed the failing test

* fix(tickets): added back attachments to sendEmail function

* fix(tickets): adapted the test to last changes
* fix(discord): solve rate limit issues

* fix(discord): take rate limit headers into account

* fix(discord): removed trailing log

* fix(discord): fetch user loop fix

* test(discord): add tests or codecov won't like my pr

* test(discord): actually enable tests
fix: correct discord channels
fix(user): users should not be able to join a team when created another one
fix(user): allow lastname to be two char length (#91)
* fix(items): added remaining item count to /items

* fix(items): correct doc and added loop for tests
@TeddyRoncin TeddyRoncin marked this pull request as draft September 4, 2024 21:04
@TeddyRoncin TeddyRoncin marked this pull request as ready for review September 8, 2024 01:40
Copy link
Contributor

@Remi-pcn Remi-pcn left a comment

Choose a reason for hiding this comment

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

Tout semble bon apart les quelques trucs qu'on a comment. @DevNono si jamais tu peux jeter un coup d'oeil aussi

src/controllers/users/createCart.ts Show resolved Hide resolved
tests/stripe/paymentAcceptedWebhook.test.ts Show resolved Hide resolved
tests/stripe/paymentAcceptedWebhook.test.ts Show resolved Hide resolved
tests/utils/pdf.test.ts Show resolved Hide resolved
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.