search

aeternity / ae_mdw

Aeternity Middleware in Elixir
26 stars 10 forks source link

fix: handle transfers txi_idx ref tuple #1794

Closed sborrazas closed 4 weeks ago

vatanasov commented 1 month ago

I think the type in the model should also be changed?

sborrazas commented 1 month ago

I think the type in the model should also be changed?

The Model type is already the correct one: https://github.com/aeternity/ae_mdw/blob/master/lib/ae_mdw/db/model.ex#L1032

vatanasov commented 1 month ago

I think the type in the model should also be changed?

The Model type is already the correct one: https://github.com/aeternity/ae_mdw/blob/master/lib/ae_mdw/db/model.ex#L1032

Right, but isn't the deserialization of the cursor also affecting the TargetKindIntTransferTx and KindIntTransferTx tables? They have txi as the fourth element in their index tuple.

sborrazas commented 1 month ago

I think the type in the model should also be changed?

The Model type is already the correct one: https://github.com/aeternity/ae_mdw/blob/master/lib/ae_mdw/db/model.ex#L1032

Right, but isn't the deserialization of the cursor also affecting the TargetKindIntTransferTx and KindIntTransferTx tables? They have txi as the fourth element in their index tuple.

You're right, those weren't adjusted. Done now. Thanks!