cardano-community / cardanosharp-koios

6 stars 6 forks source link

pool_updates: fields related to retirement aren't snapshots from the update but show current status #27

Closed robinboening closed 1 year ago

robinboening commented 1 year ago

Describe the bug Requesting pool updates returns a a list of snapshots of the values when the update was issued. Though, this is only true for most of the rows. If you request updates for a retired pool you'll notice retiring_epoch is set in every single update item, even for the initial registration of the pool, the same applies to pool_status which is filled with retired in every item.

You could argue this is intended, but I'd say it is not only very confusing as all the other columns are snapshots while these two always represent current state. It also prohibits me to detect what update is responsible for a deregistration. And, if I want to see the current status of a pool I'd request the pool_info endpoint.

To Reproduce Steps to reproduce the behavior:

  1. request pool_updates with pool1y2gynck9pd4en5rzm0rueqd2qm7sw89aqmkcntd899vv237qj38
  2. See the retiring_epoch and pool_status fields filled in all items.

Expected behavior

I expect the response to

Screenshots response for the retired pool mentioned above. It is missing 2 deregistration updates (#26) and it shows values for pool_status/retiring_epoch aren't snapshots of when the update was issued but current state.

Screenshot 2023-10-09 at 11 41 25
robinboening commented 1 year ago

Closing as this was posted in the wrong repo. Sorry for the confusion!