-
Notifications
You must be signed in to change notification settings - Fork 60
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/org bounty payment #642
Conversation
frontend/app/src/people/widgetViews/summaries/wantedSummaries/CodingBounty.tsx
Show resolved
Hide resolved
db/config.go
Outdated
hasRole = true | ||
return hasRole |
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.
can just do return true
here
hasRole = true | |
return hasRole | |
return true |
if pubKeyFromAuth != org.OwnerPubKey { | ||
userRoles := DB.GetUserRoles(uuid, pubKeyFromAuth) | ||
hasRole = RolesCheck(userRoles, role) | ||
return hasRole |
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.
can change to return false
return hasRole | |
return false |
|
||
func UserHasAccess(pubKeyFromAuth string, uuid string, role string) bool { | ||
org := DB.GetOrganizationByUuid(uuid) | ||
var hasRole bool = false |
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.
can remove this line if you do the two comments below
Describe your changes
This feature allows organizations to add budget and pay bounty from the budget
Issue ticket number and link
Type of change
Checklist before requesting a review