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

Bounty dashboard with github account verification #674

Closed
wants to merge 0 commits into from

Conversation

mvp5464
Copy link
Contributor

@mvp5464 mvp5464 commented May 18, 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

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

mvp5464 commented May 18, 2024

Confirmation message to github after successful DB. Also message for duplication of Bounty

B_confirmComment.webm

@mvp5464
Copy link
Contributor Author

mvp5464 commented May 19, 2024

Hi @devsargam @nimit9 , could you review this pr specially the GithubLogin.tsx page where user Links his account to github.
Is there any better approach for it. Or any other way to improve this pr.

@nimit9
Copy link
Contributor

nimit9 commented May 19, 2024

Hey, This might not be needed

We have a bounty-hook repo which tracks the bounties, https://github.com/code100x/bounty-hook

@mvp5464
Copy link
Contributor Author

mvp5464 commented May 19, 2024

Then should I make this changes to bounty-hook repo and remove from here.
Thank you for suggestions.

@nimit9
Copy link
Contributor

nimit9 commented May 19, 2024

Yeah, you can, but these features might not be needed.

@hkirat can answer better on this though.

@mvp5464
Copy link
Contributor Author

mvp5464 commented May 19, 2024

may be you are right, thank you

@hkirat
Copy link
Contributor

hkirat commented May 19, 2024

feature looks decent , but yeah integrate the BE with bounty-hook

@mvp5464
Copy link
Contributor Author

mvp5464 commented May 19, 2024

OK sir, working on it

@mvp5464 mvp5464 marked this pull request as draft May 19, 2024 09:59
@ebe25
Copy link

ebe25 commented May 19, 2024

hey mvp cool feature boss, are currently working change the BE to the bounty webhook?

@mvp5464
Copy link
Contributor Author

mvp5464 commented May 19, 2024

hey mvp cool feature boss, are currently working change the BE to the bounty webhook?

Thanks for the appreciation brother. Yes, getting some errors, but will solve it.

@mvp5464 mvp5464 marked this pull request as ready for review May 19, 2024 19:47
@mvp5464
Copy link
Contributor Author

mvp5464 commented May 19, 2024

@hkirat Sir, Integrated BE with bounty-hook.

Bounty comment on PR => github-webhook => bounty-hook => CMS => Confirmation comment with info on same PR

Also, created PR in bounty-hook to send data

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
4 participants