Vstam1/simplifications

[vstam1]

The following simplifications are proposed:

• Remove Candlestick Auction
• Merge Community Remainder Round
• Remove Project decision
• Remove automatic transition for transition
  • Remove on_initialize logic
  • Adjust all transition inner functions to handle transition
  • Update all root functions -> normal functions, that anyone can call to transition the project
• Rewards payout for evaluators based on total value evaluated, so removal of evaluators iterator
• Remove Y variable from rewards payout

Todo:

• Implement all simplifications
• Fix all the TODO Simplification comments
• Remove random end block
• Update tests based on simplifications
• Update benchmarks + used weights + weight returns of functions
• General cleanup
• Test with chopsticks if everything functions correctly (especially round transitions)
• Update Felix on breaking changes, so we can prepare front-end
• Create offchain worker script that queries project states and round durations and trigger transition function once round if over.
• Test on Rolimec

Important things to test:

• Does the settlement of failed bids work properly using the settle_failed_bids function. See candlestick commit
• Does refunding of either Partial bid or Bid with original_price > wap work correctly using settle_succesful bid. See candlestick commit
• Double check new wap calculation based on bucket. Already one test added.
• If all round transitions are not set up correctly, with the correct checks. Check if not possible to transition early, or call it again after round already transitioned with the new round_duration structure and generalized transition function.
• Test the merge of community + remainder round. See following lines for important changes:
  • Buy last CT in com/rem round if ticket size < min_ticket_size. here
  • Active bidder can participate after status::CommunityRound(BLOCK_NUMBER). See round transition and check
• The full new funding_end structure where project decision has been removed.

[JuaniRios]

• 🐛 Fix Compilation #364 : 2 dependent PRs (#366 , #370 )
• 🚮 Remove "Y" variable from fee allocation #363
• ✲ Transitions can be called by anyone #362
• 🧹 Cargo fmt #361
• Vstam1/simplifications #358 👈
• 😎 Runtime APIs #354
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @vstam1 and the rest of your teammates on Graphite

[JuaniRios]

Merge activity

• Aug 23, 5:56 AM EDT: @JuaniRios started a stack merge that includes this pull request via Graphite.
• Aug 23, 5:56 AM EDT: @JuaniRios merged this pull request with Graphite.