Tuba Head [Production Release] #846
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
it's not bit-packed when placed after the auction struct and for now we prefer to not force it into the auction struct esp. since bigger gas savings are coming from descriptor storage read optimizations
because we hit the contract size limit, this commit has a POC for reducing it. all the admin functions are replaced with a fallback function that delegate calls the admin lib.
prep for adding tests that need to interact with the DAO
getPrices(n)
- returns exactly `n` latest auction prices
- skips auctions with no bids
- skips nounder noun ids
- reverts if auction should have data but doesn't
- reverts if not enough items in history (not need to trim)
-> either returns array of n-length or reverts
getSettlements(n, skipEmptyValues)
- returns up to `n` latest settlement (in case we reach id 0)
- includes auction with no bids (blockTimestamp will be > 1)
- skips nounder noun ids if skipEmptyValues = true, otherwise return raw
- skip unset data if skipEmptyValues = true, otherwise return raw
getSettlements(x, y, skipEmptyValues)
- returns settlements for ids [x,y)
- includes auction with no bids (blockTimestamp will be > 1)
- skips nounder noun ids if skipEmptyValues = true, otherwise return raw
- skip unset data if skipEmptyValues = true, otherwise return raw
…nd-reduce-contract-size
…incentives-noracle
…incentives-noracle
since we had a recent incident where a candidate with more than 10 txs was created only to fail to convert to a proposal because of the tx limit proposals have
…-update-candidate-fee Data: remove candidate creation fee when used to update a proposal
moved the voting ended check into `proposalDataForRewards`. otherwise, a proposal that hasn't yet finished voting may get filtered out due to not meeting the quorum requirements and the `nextProposalIdToReward` could be set to a proposalId after it. This would cause that proposal to not get rewards.
…ix-1 Fix for rewards contract following audit
…an-set-descriptor allow admin to set rewards descriptor
separate params setting functions added a function to enable each one individually which also resets the next id to reward
update update-configs-dao-v3.ts: fix subgraph config generation
…posal splitting auction & proposal rewards
…acle Client Incentives + Noracle
Add Tuba head
✅ Deploy Preview for nouns-home ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for nouns-testnet ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.