-
Notifications
You must be signed in to change notification settings - Fork 344
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implement liveTx and min/max height search filters #102
Conversation
aab3d1e
to
1f0b684
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice
txs = await this.cosmWasmClient.searchTx({ sentFromOrTo: sentFromOrTo }); | ||
} else { | ||
throw new Error("Unsupported query"); | ||
} | ||
|
||
return txs.map(tx => this.parseAndPopulateTxResponseUnsigned(tx)); | ||
const filtered = txs.filter(tx => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oof, filtering client side. "Fake it til you make it".
But yeah, I agree you can fulfill the API properly now, while not being blocked on the "proper" backend changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Exactly. But feel free to bring in CosmWasm/wasmd#73
throw new Error("Query by minHeight/maxHeight not supported together with ID"); | ||
} | ||
|
||
// concat never() because we want non-completing streams consistently |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
interesting
* | ||
* @see https://cosmos.network/rpc/#/Transactions/post_txs | ||
*/ | ||
export enum BroadcastMode { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good docs
No description provided.