Insrt-Finance / insrt-v2-contracts

0 stars 3 forks source link

Add state machine to ensure no depositors can exploit unfulfilled mints #138

Closed NouDaimon closed 1 year ago

NouDaimon commented 1 year ago
NouDaimon commented 1 year ago

Realize this is not yet complete because of this comment: https://github.com/Insrt-Finance/insrt-v2-contracts/issues/121#issuecomment-1687986505

I think adding it now would increase complexity a lot. Shall we split into 2 issues and tackle separately?

Also please give extra care to the fulfillRandomWords test edits.

CruzMolina commented 1 year ago

Realize this is not yet complete because of this comment: #121 (comment)

I think adding it now would increase complexity a lot. Shall we split into 2 issues and tackle separately?

Agree on this.