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.

Sign up for GitHub

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

Jump to bottom

optional withdrawer #75

Merged

bekauz merged 3 commits into main from benskey/optional-withdrawer

Jul 22, 2023

Collaborator

bekauz commented Jul 22, 2023

Closes #71

Holder can now be instantiated without a withdrawer address
WithdrawLiquidity and Withdraw methods will now return a NoWithdrawerError in case no withdrawer is set
Withdrawer can be updated via migration update

bekauz added 3 commits

July 22, 2023 12:52


          optional withdrawer

99591f7


          extra response attributes

1649b42


          unset withdrawer address unit tests

185394b

Contributor

uditvira commented Jul 22, 2023

looks good!

bekauz merged commit da6aa15 into main

bekauz deleted the benskey/optional-withdrawer branch

August 9, 2023 15:59

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet