search

CKS-Systems / manifest

The Unlimited Orderbook
https://www.manifest.trade
GNU General Public License v3.0
32 stars 17 forks source link

make ref account optional in settle funds #262

Closed abrzezinski94 closed 2 weeks ago

brittcyr commented 2 weeks ago

This needs a change in the idl, I can do that in a couple hours

abrzezinski94 commented 2 weeks ago

This needs a change in the idl, I can do that in a couple hours

Are you sure it’s already optional in rust?

brittcyr commented 2 weeks ago

Contributor

Looks like it was supposed to be https://github.com/CKS-Systems/manifest/blob/77e51df363ff72c2b713bd1995f713c4dd104b21/programs/ui-wrapper/src/instruction.rs#L115 but wasnt done correctly for idl

program is supposed to handle it https://github.com/CKS-Systems/manifest/blob/77e51df363ff72c2b713bd1995f713c4dd104b21/programs/ui-wrapper/src/processors/settle_funds.rs#L67

brittcyr commented 2 weeks ago

Doing it here https://github.com/CKS-Systems/manifest/pull/264