wlmyng
wlmyng
@amnn since we have your cursor framework, I'll close this PR so we can use that instead
> Note that the `StoredTransaction` keeps track of a serialized representation of all its events: > > https://github.com/MystenLabs/sui/blob/0a4182d17651e5b4d496f8ca2fa4610362bcf9af/crates/sui-indexer/src/models_v2/transactions.rs#L36 > > So it would be better to implement this referencing that,...
> > I think I can update Event representation to have optional StoredEvent and required NativeSuiEvent > > > > I was just having a look at this -- there...
Ready for another review. Figured it'd be easier to just do the "right" thing straight up (use serialized events on StoredTransaction)
> @manolisliolios I probably lack a lot of context, but curious why `test.test.sui` would be accepted: isn't that. a subdomain of `test.sui`? And if yes, how does it work if...
@amnn i think this might need a rebase?
> Will prepare the stack for landing based off this PR! and so it begins! exciting
> Will error rate on Grafana still count the new error? Just verified. This is technically a success response to jsonrpc, and so in the metricsLogger.on_result the success bool is...
This endpoint is available on the rpc 2.0 implementation in graphql. Example `.graphql`: ``` { objects(filter: {type: "0x2::coin::Coin", owner:"0xaddress"}) { nodes { address } } } ```
Okay, looks like it ultimately comes down to just an indexer db schema change.