search

hats-finance / hats-contracts

MIT License
36 stars 47 forks source link

createVault params should be named params #464

Closed jellegerbrandy closed 1 year ago

jellegerbrandy commented 1 year ago

We'd like to remove the HATVAultInitParams struct in favor of named arguments - for reasons of UX.

To avoid the stack too deep error, we can remove the rewardControllers list, and if necessary also the isPaused param (default is "paused")

jellegerbrandy commented 1 year ago

closing bc it is not feasible with the memory limitatations of solidity