feat/add razorpay #323
feat/add razorpay #323
Conversation
|
Great work buddy, reviewing the code right now check your discord dm lets hop on a call too |
|
fix the build and conflicts please |
|
Build errors still exist, please solve them ... need to merge this asap |
|
You'll have to add env variables for Razorpay in github actions |
| } | ||
| data.companyLogo = (await submitImage(file)) ?? ''; | ||
| await processPayment({ | ||
| amount: 1000, |
| return; | ||
| } | ||
|
|
||
| const order = await fetch('/api/order', { |
There was a problem hiding this comment.
can we turn these in actions ?
| }, | ||
| body: JSON.stringify({ | ||
| amount: (amount * 100).toString(), | ||
| currency: 'INR', |
There was a problem hiding this comment.
are we not allowing other currencies ?
|
|
||
| const intializeRazorpay = () => { | ||
| return new Promise((resolve, reject) => { | ||
| const script = document.createElement('script'); |
There was a problem hiding this comment.
use a next-script tag , it'll be cleaner (refer @ui//gmaps-autosuggest
| <Button className="w-full rounded-full mt-4"> | ||
| Continue to Payment | ||
|
|
||
| <Button |
There was a problem hiding this comment.
can we put this button inside the form as a submit
| ); | ||
| } | ||
|
|
||
| const jobData = jsonBody.payload.payment.entity.notes; |
There was a problem hiding this comment.
we have lodash library in deps , lets leverage the _.get function to access nested values
| } | ||
|
|
||
| const transactionData = { | ||
| razorpayOrderId: jsonBody.payload.payment.entity.order_id, |
| { status: 200 } | ||
| ); | ||
| } catch (error) { | ||
| return NextResponse.json({ error }, { status: 500 }); |
There was a problem hiding this comment.
we're serializing each api reponse to a certain shape , refer jobs/actions.ts
|
hey is this issue still up? can i work on it? |
|
not for now we dont need it anymore |
Resolve Issue #306
@VineeTagarwaL-code
Untitled.design.mp4
Special Thanks to @ItsFlash10⚡