-
Notifications
You must be signed in to change notification settings - Fork 62
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
remove assign to self button #921
remove assign to self button #921
Conversation
Okay I see you removed the button. Let me talk with the team about whether we need to remove or comment out the code too. |
Hey @Marvel-Ib. We also would like to comment out the api call in the backend as well so that the api won't get exploited. In other words we want the code to still be there but not active. Are you familiar with go? If so, can you add that to this PR? |
I also assigned this 100k sat bounty to you: https://community.sphinx.chat/bounty/685 |
@elraphty Hi, please can you help review this and point out if there is anywhere else in the go code I need to remove |
@elraphty still waiting for a review |
@Marvel-Ib in tribes.go GenerateInvoice function Tribes.go remove these
and also remove the ASSIGN conditional statement from bounty.go
|
@Marvel-Ib we are going to give you partial payout of the bounty for this. We need to get this addressed so Raph will be taking it over. |
Describe your changes
remove assigning to self functionality
Screen.Recording.2023-11-10.at.05.01.20.mov
Issue ticket number and link
#918
Type of change
Please delete options that are not relevant.
Checklist before requesting a review