Skip to content

Conversation

@0xSasaPrsic
Copy link
Member

@0xSasaPrsic 0xSasaPrsic commented Apr 1, 2025

Add receiver address transactions that match the query.
Add a var to control the limit of the query.

@abheektripathy
Copy link
Member

hey @0xSasaPrsic are we checking for duplicates before returning?

@0xSasaPrsic
Copy link
Member Author

hey @0xSasaPrsic are we checking for duplicates before returning?

There is no duplicate check.

@abheektripathy
Copy link
Member

@0xSasaPrsic we should add one, before returning imo, lemme know if you'd want to understand, how there will be duplicates in the first place

Copy link
Member

@QEDK QEDK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs better pagination, please remove immutable from cache header when the data can change in the future.

let ethereum_sends_results = eth_send_query
.select(EthereumSend::as_select())
.order_by(schema::ethereum_sends::source_timestamp.desc())
.limit(state.transactions_result_max_size.into())
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please implement pagination with offset and integrate with endpoint.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This simplifies ui

let avail_sends_results = avail_send_query
.select(AvailSend::as_select())
.order_by(schema::avail_sends::source_timestamp.desc())
.limit(state.transactions_result_max_size.into())
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same as above.

@0xSasaPrsic 0xSasaPrsic force-pushed the expand_transaction_query_result branch from 3209cf9 to 7c3c6e5 Compare June 10, 2025 11:49
@0xSasaPrsic 0xSasaPrsic force-pushed the expand_transaction_query_result branch from 7c3c6e5 to 869da22 Compare June 10, 2025 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants