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
multi: Add billing statuses. #1480
Conversation
@lukebp Thanks for your valuable review, updated. |
setting billing status
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.
This is looking good. I left some comments, but they're mostly formatting/documentation related. Nice work.
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.
tACK
A test case for the vote status verification and the success test case were left off because a testing framework has not been added yet for backend calls. I'm working on this. We can go back and fill in these two tests once I've added it. Test cases for all other validation are included in this PR.
This cannot be merged until the v1.1.0 release has been cut.
This diff adds billing statuses to
polteiad
'spi
plugin and topoliteiawww
'spi
api.Admins are able to set a billing status of an approved proposal using
the new
pictl
command:proposalsetbillingstatus "token" "status" "reason"
Note: reason is required only when setting record's billing status
to
closed
.Currently, It's allowed to set a billing status only once.
Closes #1474.