-
Notifications
You must be signed in to change notification settings - Fork 3
Create operating fund #67
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
base: main
Are you sure you want to change the base?
Conversation
STX and FT will now automatically go to OP_FUND based on period and logic defined in next commits. NFT stays the same for now.
Need to verify that transfers are not possible outside these funcitons with tests and the treasury is officially locked down on an available 30day drip to the operating fund.
So with other contracts we have a pattern of STX, sBTC, and DAO token. In the treasury we have STX, FT, and NFT in a more broad definition. If we track the claims per period for just FT, then we could only withdraw sBTC or DAO token once per period, no separation. I think if we focus on the treasury as being the secure place for the DAO token, STX, and sBTC then we can scope it better then just being a general treasury. The question becomes what if another FT or NFTs get added? Do we need to support that now or introduce something later? To sum it up in a different way, should we streamline it so that:
|
List support is nice but let's keep this as simple as possible. List can be in the proposal as lines of code instead of a literal list, and we're not expecting a long list of entries that would push the limits anytime soon.
We might end up needing a blank/empty test file like this for each depending on where it was used, updated to just v3 for now to clear clarinet check errors.
Also more specific on print statement contents and consistency to use in other contracts.
Also updates error codes in prep for new extension, groups action errors with action proposals.
Should help with a few checks we're doing in separate contracts.
Considering splitting this into three distinct contracts with a specific purpose.
This PR adds the operating fund contract
This also modifies the treasury to interact solely with the operating fund, creating an extra layer of protection and allowing us to open up core proposals sooner than originally anticipated in #39
Left to do:
Notable Changes:
get-current-period
,get-stx-claim
andget-ft-claim
get-contract-info
with constants and configupdate-stx-claim
,update-ft-claim
with write-once logic