-
-
Notifications
You must be signed in to change notification settings - Fork 775
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
feat: ability to suggest bounties via tribes #5927
Conversation
- create bounty request flow - display suggested bounties - reject bounty requests
Codecov Report
@@ Coverage Diff @@
## master #5927 +/- ##
=======================================
Coverage 17.44% 17.44%
=======================================
Files 259 259
Lines 23305 23305
Branches 3396 3396
=======================================
Hits 4065 4065
Misses 19230 19230
Partials 10 10 Continue to review full report at Codecov.
|
@@ -389,6 +390,13 @@ def create_new_bounty(old_bounties, bounty_payload, bounty_details, bounty_id): | |||
else: | |||
raise UnsupportedSchemaException('No webReferenceURL found. Cannot continue!') | |||
|
|||
try: | |||
bounty_request = BountyRequest.objects.get(github_url=url, status='o') | |||
bounty_request.status = 'f' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is that status 'f' ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
status f -> is status funded
@@ -47,7 +47,7 @@ | |||
{% endif %} | |||
</div> | |||
|
|||
<div class="container-fluid bg-light"> | |||
<div class="container-fluid"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removing bg-light will turn the bg white right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM just left some comments
Description
TRIBE MEMBER
TRIBE OWNER
Also, introduces 2 new REST Endpoints with validation for bounty requests
Refers/Fixes