Innovateninjas / Paws-frontend

A fast responsive webapp that allows you to report injured stray animals to the nearest NGOs.
https://pawss.vercel.app
GNU General Public License v3.0
63 stars 101 forks source link

Fixes#304: Payment gateway issue fixed #439

Open SaurabhS55 opened 5 months ago

SaurabhS55 commented 5 months ago

Hey @codewarnab, I have fixed payment gateway issue for desktop/laptop users using Razourpay payment gateway. Currently, the gateway is in test mode. As you said previous feature only works on mobile not on desktop, By considering all this issues, I have implemented following features :-

  1. Previous payment option is also available for mobile handset users, and for desktop users I implemented razorpay. I controlled both options by simpley applying check on screen size.
  2. Current payment feature is on test mode, to make it live just replace my test secret keys with you live keys.
  3. Unlike stripe, Razourpay requires very few docs, PAN number, mobile number and KYC for live link.

before payment gateway:- image

After payment gateway:- image

SaurabhS55 commented 5 months ago

@codewarnab review the changes once.

codewarnab commented 5 months ago

update the env.example for configuration also and share any other documenetation we might needed for the proper setup

codewarnab commented 5 months ago

what did you changed in nav bar ? also mention the issue that the pr correspons to

SaurabhS55 commented 5 months ago

what did you changed in nav bar ? also mention the issue that the pr correspons to

I haven't changed anything in navbar. Issue No. :- 304

SaurabhS55 commented 5 months ago

update the env.example for configuration also and share any other documenetation we might needed for the proper setup

To create live credentials for Razorpay, you can generate API keys in live mode on your Razorpay dashboard:

  1. Log in to your Razorpay account
  2. Navigate to Account & Settings → API Keys → Generate Key
  3. Download the keys and save them securely
  4. Replace the test API key with the live key

Following Documents might required for KYC to generate live link :-

  1. Business PAN card details
  2. Adhar card details
codewarnab commented 5 months ago

@rishicds review and test this pr thorougly and then merge this

SaurabhS55 commented 5 months ago

Hey @codewarnab @rishicds what is status of review?

rishicds commented 5 months ago

Hi @SaurabhS55 , we have exams currently,rest assured I'll review it soon and merge it when I'm free,till then you can work on another issue

SaurabhS55 commented 4 months ago

Hi @SaurabhS55 , we have exams currently,rest assured I'll review it soon and merge it when I'm free,till then you can work on another issue

Hey @rishicds @codewarnab any update on review?

rishicds commented 4 months ago

@SaurabhS55 hi I checked its okay,good job,if possible please fix the merge conflicts

SaurabhS55 commented 4 months ago

@rishicds Resolved conflicts !! plzz review the changes once before merging.

codewarnab commented 4 months ago

it will cause issues we just moved our project to Vite from CRA it will be better if you just create a fresh fork and put your chanages their, rather than updating the fork . will not be much work i think