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.

Sign up for GitHub

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

Jump to bottom

Fixes#304: Payment gateway issue fixed #439

Open
wants to merge 89 commits into
base: main
Choose a base branch
from
Open

Fixes#304: Payment gateway issue fixed #439

wants to merge 89 commits into from

Conversation

SaurabhS55
Copy link

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

suhanipaliwal and others added 30 commits May 11, 2024 20:27
@suhanipaliwal
Create CODE_OF_CONDUCT.md
ebe034c
@adarsh1001111
The firebase google auth is added and is functional
cd2a52f
@adarsh1001111
The functionality tested and the api keys are restored to the origina…
182965e 
…l of the paws project
@adarsh1001111
Update firebase-messaging-sw.js
7c8041a 
removed unnecessary commented part.
@adarsh1001111
Merge branch 'main' into googleAuth
0b422db
@codewarnab
modified: .gitignore
5fd6d74
@rishicds
Merge pull request Innovateninjas#289 from suhanipaliwal/suhanipaliwa…
94fd731 
…l-patch-3

Added CODE_OF_CONDUCT.md
@rishicds
Merge pull request Innovateninjas#290 from NilanchalaPanda/fix/Remove…
e6761b8 
…ModuleFile

[FIX] - Module.css files removed and Tailwind CSS applied.
@adarsh1001111
Update index.html all unneccessary "/" removed
c38064b
@adarsh1001111
Update index.html
f31fa9e
@adarsh1001111
unncessary imports removed
c98b3f4
@adarsh1001111
Merge branch 'Innovateninjas:main' into googleAuth
443dd4e
@Tanishka023
Add files via upload
b36c91c
@adarsh1001111
Update .env.sample
a745fca
@codewarnab
Update CONTRIBUTING.md
be6b1a5
@codewarnab
Update CONTRIBUTING.md
44e08f7
@Tanishka023
Merge branch 'main' into tanishka_favicon
15cbcaa
@Priyal208
Improve UI of login page
e875514
@Priyal208
Merge branch 'main' of https://github.com/Priyal208/Paws-frontend-priyal
4362a01
@vasundhara943
Added nearest NGO info on success page
6aa167c
@vasundhara943
Merge branch 'Innovateninjas:main' into main
411448c
@sanmarg
Create greetings.yaml
d29087c 
Added For Innovateninjas#316
@rishicds
Merge pull request Innovateninjas#300 from Tanishka023/tanishka_favicon
489330a 
Adds new favicon
@rishicds
Merge pull request Innovateninjas#292 from adarsh1001111/googleAuth
37dc45f 
Google auth
@rishicds
Merge pull request Innovateninjas#328 from sanmarg/patch-1
5d9fa2a 
Create greetings.yaml
@rishicds
Merge pull request Innovateninjas#318 from vasundhara943/main
9515ea2 
Added Nearest NGO Info in Success Page
@codewarnab
Update CONTRIBUTING.md
cab612c
@codewarnab
Update CONTRIBUTING.md
24ffd78
@Rohitha-pudu
install bootstrap icons
00d4b7f
@Rohitha-pudu
update package-lock.json
d2d1876
@codewarnab
Copy link
Collaborator

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

codewarnab added 3 commits June 6, 2024 00:02
@codewarnab
deleted: favicon (2).ico
aaeace6
@codewarnab
modified: .gitignore
62f8b73 
	deleted:    public/images/search.png
	modified:   src/firebase.js
	modified:   src/utils/Functions/userAuthService.js
@codewarnab
Merge branch 'main' of https://github.com/Innovateninjas/Paws-frontend
4697f3b 
ok
@codewarnab codewarnab mentioned this pull request Jun 5, 2024
Open
@MohanRamSridhar @rishicds
Create sync-fork-workflow.yml (Innovateninjas#459)
2bbbd54 
* Create sync-fork-workflow.txt

* Rename sync-fork-workflow.txt to sync-fork-workflow.yml

---------

Co-authored-by: Rishi <[email protected]>
@SaurabhS55
Copy link
Author

Hey @codewarnab @rishicds what is status of review?

@rishicds
Copy link
Collaborator

rishicds commented Jun 8, 2024

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
Copy link
Author

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
Copy link
Collaborator

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

@rishicds rishicds added level3 45 points gssoc Girl Script Summer of Code labels Jun 22, 2024
RamakrushnaBiswal and others added 8 commits June 22, 2024 14:49
@RamakrushnaBiswal @rishicds @codewarnab
Loader added for the website
fa88e64 
* Loader added for main page

* Update App.jsx

---------

Co-authored-by: Rishi <[email protected]>
Co-authored-by: Arnab Mondal <[email protected]>
@Priyal208 @codewarnab
landing page figma file added
e62a12b 
Co-authored-by: Arnab Mondal <[email protected]>
@codewarnab
fix
4b337bf
@SaurabhS55
Merge branch 'main' of https://github.com/SaurabhS55/Paws-frontend
05362b7
@the-t3ch-wizard @codewarnab
Register and Login page responsive (Innovateninjas#462)
dbc2c4b 
* fixed conflicts

* improved responsive ui of register and login page

* removed duplicate keys

* added navigation link of register and login as NGO

---------

Co-authored-by: Arnab Mondal <[email protected]>
@SaurabhS55
conflicts resolved
bdb31e4
@SaurabhS55
Fixes:Innovateninjas#304 branch conflicts resolved
50795c1
@SaurabhS55
Merge branch 'main' into paymentGateway
a450cba
@SaurabhS55
Copy link
Author

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

@codewarnab
Copy link
Collaborator

codewarnab commented Jun 22, 2024 •
edited
Loading

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@github-actions github-actions[bot] github-actions[bot] left review comments

@rishicds rishicds Awaiting requested review from rishicds

Assignees

@SaurabhS55 SaurabhS55

Labels
gssoc Girl Script Summer of Code level3 45 points
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.