Roman

Results 22 comments of Roman

good work. couple of comments: 1) for future extensibility we have to consider all possibilities in blockchain realities https://en.wikipedia.org/wiki/Order_(exchange) 2) `OrderBookOrderPlacement` and `OrderBookMarketOrders` are those modules? not sure I understood...

Hi @raulmonge thanks for reporting this. the issue is making a loop from 0 to 300 exceed block gas limit or free infura block gas limit. you can either run...

btw. previosly this was working but cloud providers making this more and more strict to save on resources

actually there is an issue with BZRX loan as collateral. when there is no collateral left. we plan to fix this

@yaronvel I think this is the right idea for a POC. I think we can move on with the implementation

rebase this to itoken-collateral, we might wanna merge it before we deploy itoken-collateral. since ProtocolSettings is changed

this needs to be rebased to itoken-collateral. what do we need to do to include this in arbitrum deployment?

this looks good. I would love to se a formal tech document describing processes that are going to happen. also will be usefull for a blog post

Hi @HighBe , we are aware of the issue. we might implement it next time we upgrade the DAO. however right now it doesn't bring us immediate benefit.