Sifchain / sifnode

SifNode - The future of Defi
Other
108 stars 119 forks source link

Litepaper Orderbook Section #2867

Open Brando753 opened 2 years ago

Brando753 commented 2 years ago

Missing implementation for order books In the Sifchain Lite Paper, it is mentioned that "Sifchain uses both an order book and a CLP for trade completion". However, the code base does not have any implementation related to order books. Either todo's should be added to support orderbooks, or the paper should be updated.

pandaring2you commented 2 years ago

@jkbrooks - do we want to remove this from the litepaper? or just put in a disclaimer that this is aspirational? cc @jon-sif

pandaring2you commented 1 year ago

from Jazear: ah then yea let's just deprecate the Litepaper for this audit peggy.sifchain.finance is the only definitive document

Jordan: If they ask for high level documentation that the litepaper covers that peggy.sifchain.finance doesnt cover, I suppose we can add the CLP and IBC sections to docs.sifchain.finance

pandaring2you commented 1 year ago

Jazear: "you should definitely tell Certik that the definitive docs are https://peggy.sifchain.finance" . @sifmoon - can we deprecate this portion of the litepaper to satisfy certik?

sifmoon commented 1 year ago

Yeah sure. Is this something I need to assist with, or are you good to go in doing this?

pandaring2you commented 1 year ago

@sifmoon - yeah - canyou remove this portion from the litepaper or note that it is aspirational? however you wanted to do this so that certik can say that what we're advertising in the litepaper is what we're delivering