-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Conversation
Confirmation message to github after successful DB. Also message for duplication of BountyB_confirmComment.webm |
Hi @devsargam @nimit9 , could you review this pr specially the GithubLogin.tsx page where user Links his account to github. |
Hey, This might not be needed We have a bounty-hook repo which tracks the bounties, https://github.com/code100x/bounty-hook |
Then should I make this changes to bounty-hook repo and remove from here. |
Yeah, you can, but these features might not be needed. @hkirat can answer better on this though. |
may be you are right, thank you |
feature looks decent , but yeah integrate the BE with bounty-hook |
OK sir, working on it |
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. |
@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 |
PR Fixes:
Resolves #673
Checklist before requesting a review
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
Need to add email to read-only for github provider (so we can get email (even if hidden) when user signin with github