Skip to content
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

Use INX listen_to_treasury_updates instead of blocks.rs #597

Open
Tracked by #685
grtlr opened this issue Aug 25, 2022 · 2 comments
Open
Tracked by #685

Use INX listen_to_treasury_updates instead of blocks.rs #597

grtlr opened this issue Aug 25, 2022 · 2 comments
Labels
waiting Issues that are blocked right not

Comments

@grtlr
Copy link
Contributor

grtlr commented Aug 25, 2022

This should be cleaner and better for performance because we don't have to filter over all Blocks of a milestone. It would probably be best to do something similar like we do with the ledger updates:

cx.add_stream(LedgerUpdateStream::new(
    inx.listen_to_ledger_updates((start_index.0..).into()).await?,
));
@grtlr grtlr added the bug Something isn't working label Aug 25, 2022
@grtlr grtlr changed the title Use INX listen_to_treasury_updates instead of piggybacking Blocks Use INX listen_to_treasury_updates instead of blocks.rs Aug 25, 2022
@grtlr
Copy link
Contributor Author

grtlr commented Oct 20, 2022

We are waiting for a decision on the future of the Treasury output.

@grtlr grtlr added the waiting Issues that are blocked right not label Oct 20, 2022
@DaughterOfMars
Copy link
Collaborator

Still waiting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
waiting Issues that are blocked right not
Projects
None yet
Development

No branches or pull requests

2 participants