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

Added Bounty dashboard + Page to Link github account #686

Closed
wants to merge 11 commits into from

Conversation

mvp5464
Copy link
Contributor

@mvp5464 mvp5464 commented May 20, 2024

PR Fixes:

  • 1 Bounty dashboard for users and admin
  • 2 Link github account with the cms account
  • 3 Listen to bounty message with github webhook

Resolves #673

Checklist before requesting a review

  • I have performed a self-review of my code
  • I assure there is no similar/duplicate pull request regarding same issue

info

New UI with added features

UI.webm

Linking github account

B_userLogin.webm

Different dashboard for both User and admin

B_adminUser.webm

Listen to bounty message and update to database

B_addBounty.webm

All Features

  • One bounty per PR (to avoid duplicate bounty)
  • Users can link their 100xdevs account with their github to get all bounty info
  • If account is not linked still bounty info will be added to DB
  • Admin can see them and when account gets linked all bounty info will be shown to that user
  • Two different dashboard for user and admin
  • Admin can check user info and bounty info for any user
  • user and admin can see total bounty PR and Total amount of such bounty
  • Conversion from USD to Rupee (Change daily)
  • Can support pay on click by adding payment info of users and payment gateway
  • Get information about is bounty paid or not

Need to add email to read-only for github provider (so we can get email (even if hidden) when user signin with github

Screenshot

@mvp5464 mvp5464 changed the title Bounty dashboard with github account verification Bounty dashboard with github account verification [duplicate - resolve conflicts] May 20, 2024
prisma/schema.prisma Show resolved Hide resolved
src/app/api/github-webhook/route.ts Show resolved Hide resolved
src/app/api/github/route.ts Outdated Show resolved Hide resolved
@mvp5464
Copy link
Contributor Author

mvp5464 commented May 22, 2024

@NawinKumarSharma , It would be great if you can also review the PR in bounty-hook along with this. Thank you

@mvp5464
Copy link
Contributor Author

mvp5464 commented May 26, 2024

@hkirat Sir , Added payment info page with payment facility with added features.

  • tried razorpay to pay from razorpay account to upi but it seems no easy way to do that then tried this
  • Admin have to scan and pay then click on pay button to add the payment info to DB

To add- AWS S3 details
Doubt :

  1. Integrated backend with bounty-hook as suggested by you, Is it good or need changes in Integration (PR created in bounty-hook for the same.
  2. Is revoked payment (paid to not paid) functionality needed then can also add transaction details to DB or in notion

Screenshot from 2024-05-26 09-05-20

Adding payment info by User

B_paymentInfo.webm

Payment by Admin

B_payment.webm

@mvp5464 mvp5464 changed the title Bounty dashboard with github account verification [duplicate - resolve conflicts] Added Bounty dashboard + Page to Link github account + Add payment Info + Do payment May 26, 2024
@mvp5464 mvp5464 changed the title Added Bounty dashboard + Page to Link github account + Add payment Info + Do payment Added Bounty dashboard + Page to Link github account + Add payment Info page + Payment functionality May 26, 2024
@hkirat
Copy link
Contributor

hkirat commented May 31, 2024

No need to add any payment related things here, that'll be handled via crypto
The UI looks very janky are u planning to build on top of it?

@mvp5464
Copy link
Contributor Author

mvp5464 commented May 31, 2024

No need to add any payment related things here, that'll be handled via crypto The UI looks very janky are u planning to build on top of it?

Sure Sir, Removing the payment related info like getting payment info from user, etc
Yes sir, It's a basic UI, and plan to enhance and polish it further.

@mvp5464 mvp5464 marked this pull request as draft May 31, 2024 21:29
@mvp5464 mvp5464 marked this pull request as ready for review June 1, 2024 10:46
@mvp5464
Copy link
Contributor Author

mvp5464 commented Jun 1, 2024

  1. Removed payment related info.
  2. Created new UI from scratch using shadcn and made it responsive
  • Sir @hkirat , any further improvements required ?
UI.webm

@mvp5464 mvp5464 changed the title Added Bounty dashboard + Page to Link github account + Add payment Info page + Payment functionality Added Bounty dashboard + Page to Link github account Jun 1, 2024
@Karthik73965
Copy link

bro this is'nt merged you did a lot !!

@mvp5464
Copy link
Contributor Author

mvp5464 commented Jun 16, 2024

bro this is'nt merged you did a lot !!

Thanks brother, As @hkirat sir have limited time, but now maybe @siinghd can look into this

Copy link

@Chandraprakash-Darji Chandraprakash-Darji left a comment

Choose a reason for hiding this comment

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

I am not the maintainer here, but I came across the PR. So I just reviewed your changes. You should wait for maintainer before making changes on based of my review.

src/app/api/github/route.ts Outdated Show resolved Hide resolved
src/components/GithubLogin.tsx Outdated Show resolved Hide resolved
src/lib/auth.ts Show resolved Hide resolved
@mvp5464
Copy link
Contributor Author

mvp5464 commented Jul 2, 2024

I am not the maintainer here, but I came across the PR. So I just reviewed your changes. You should wait for maintainer before making changes on based of my review.

Thanks for reviewing and valuable suggestions. will surly improve on them.

@mvp5464
Copy link
Contributor Author

mvp5464 commented Jul 8, 2024

@siinghd Sir, can you take a look here

@devsargam
Copy link
Collaborator

A lot of conflicts but you did a lot closing. But feel free to open a new one

@devsargam devsargam closed this Sep 24, 2024
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.

feature: bounty dashboard
6 participants