search

mrgnlabs / marginfi-v2

Apache License 2.0
147 stars 61 forks source link

cli: add-bank-with-seed command #158

Closed exdx closed 6 months ago

exdx commented 6 months ago

Adds a new CLI command to exercise the functionality added in #154. Will be merged alongside core changes.

jkbpvsc commented 6 months ago

I accidentally closed this one because I merged the 154-single-signer-add-bank-ix branch. Update this PR to merge to main and reopen it.

jkbpvsc commented 6 months ago

Also would probably be better to just modify the existing command to add the bank, by adding a --with-seed bool flag, the instruction handler should then search for a free address with canonical seeds.

exdx commented 6 months ago

Also would probably be better to just modify the existing command to add the bank, by adding a --with-seed bool flag, the instruction handler should then search for a free address with canonical seeds.

Sure, totally agree. I'm working on that locally. Will push it up.