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: User is now able to perform payments #127

Merged
merged 3 commits into from
Jul 26, 2024

Conversation

EddyShimwa
Copy link
Contributor

@EddyShimwa EddyShimwa commented Jul 24, 2024

What does this PR do?

  • This PR allows users to pay via momo and stripe

Description of the task to be completed

  • Allow users tomake payments on their products

What are the relevant pivotal trackers?

Copy link

netlify bot commented Jul 24, 2024

Deploy Preview for dynamite-frontend ready!

Name Link
🔨 Latest commit bf4e576
🔍 Latest deploy log https://app.netlify.com/sites/dynamite-frontend/deploys/66a3b523dee3bc00082a9fab
😎 Deploy Preview https://deploy-preview-127--dynamite-frontend.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@EddyShimwa EddyShimwa force-pushed the feat-integrate-payments branch 3 times, most recently from 4207626 to b880481 Compare July 24, 2024 11:03
@EddyShimwa EddyShimwa force-pushed the feat-integrate-payments branch 5 times, most recently from d65fd4a to 05ac433 Compare July 26, 2024 13:43
@codecov-commenter
Copy link

codecov-commenter commented Jul 26, 2024

Codecov Report

Attention: Patch coverage is 46.77419% with 33 lines in your changes missing coverage. Please review.

Project coverage is 85.70%. Comparing base (362caba) to head (229778a).

Files Patch % Lines
src/features/Checkout/checkoutSlice.ts 26.82% 30 Missing ⚠️
src/features/Cart/cartSlice.tsx 85.71% 3 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #127      +/-   ##
===========================================
- Coverage    85.91%   85.70%   -0.22%     
===========================================
  Files          101      101              
  Lines        11780    11829      +49     
  Branches       827      828       +1     
===========================================
+ Hits         10121    10138      +17     
- Misses        1657     1689      +32     
  Partials         2        2              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@EddyShimwa EddyShimwa added ready for review The PR is ready for review to be mergrd and removed WIP labels Jul 26, 2024
@EddyShimwa EddyShimwa force-pushed the feat-integrate-payments branch from 05ac433 to 5b99673 Compare July 26, 2024 13:54
Copy link
Contributor

Choose a reason for hiding this comment

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

this should be in the Public folder

Copy link
Contributor

Choose a reason for hiding this comment

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

this should be in the Public folder

@13XAVI 13XAVI self-requested a review July 26, 2024 14:07
@EddyShimwa EddyShimwa force-pushed the feat-integrate-payments branch from 229778a to cff426e Compare July 26, 2024 14:14
@EddyShimwa EddyShimwa force-pushed the feat-integrate-payments branch from 3972b38 to c3a4e84 Compare July 26, 2024 14:25
@EddyShimwa EddyShimwa force-pushed the feat-integrate-payments branch 4 times, most recently from 75850e3 to f373af5 Compare July 26, 2024 14:36
@EddyShimwa EddyShimwa force-pushed the feat-integrate-payments branch from f373af5 to bf4e576 Compare July 26, 2024 14:39
@dushimeemma dushimeemma merged commit e56d49d into develop Jul 26, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Front-End ready for review The PR is ready for review to be mergrd
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants