babylonlabs-io / babylon-staking-indexer

Other
1 stars 2 forks source link

Investigate inception bbn timestamp needed for UX #47

Closed gusin13 closed 2 days ago

jrwbabylonlab commented 2 weeks ago
type IndexerDelegationPagination struct {
    StakingTxHashHex string `json:"staking_tx_hash_hex"`
    StartHeight      uint32 `json:"start_height"`
}

once the timestamp value has been added in the indexer db for delegation collection, the pagination token logic in the API service shall also be updated. Right now the StartHeight is 0 for EOI as there is no such tx exist in BTC yet. We should probably sort by the BBN height or timestamp instead.

i.e we shall save both BBN height and its timestamp in delegation collection @gusin13

fyi @jeremy-babylonlabs

gusin13 commented 1 week ago

sure can save both, basically we need eoi_bbn_height eoi_bbn_timestamp

where inception means EOI is created in babylon

jrwbabylonlab commented 5 days ago

@gusin13 i think this issue has already been taking care of from your latest api PR, right?

jrwbabylonlab commented 5 days ago

Please ping in team channel once backend has been released as we need to apply FE changes

gusin13 commented 2 days ago

https://github.com/babylonlabs-io/babylon-staking-indexer/pull/59 https://github.com/babylonlabs-io/staking-api-service/blob/4ba5e0c683f681d07554d125dc6aafbae4fc645b/internal/v2/service/delegation.go#L21-L22

https://babylonlabsworkspace.slack.com/archives/C07DYV8TEDD/p1732684360871129

this is completed