Skip to content

fix: metrics previously did not wait for confirmation #77

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

Merged
merged 1 commit into from
May 8, 2025

Conversation

prestwich
Copy link
Member

@prestwich prestwich commented May 7, 2025

fix: updates the metrics task to use a pending tx watcher instead of get_receipt. this will allow it to wait for the transaction to confirm

@prestwich prestwich force-pushed the prestwich/normalize-tracing branch from b6e4c46 to fb9f608 Compare May 8, 2025 12:38
@prestwich prestwich force-pushed the prestwich/fix-metrics branch 2 times, most recently from aa49dc4 to f6843d0 Compare May 8, 2025 12:41
@prestwich prestwich force-pushed the prestwich/normalize-tracing branch from fb9f608 to 71ea1d3 Compare May 8, 2025 12:41
@prestwich prestwich force-pushed the prestwich/fix-metrics branch from f6843d0 to 2f7b513 Compare May 8, 2025 12:46
@prestwich prestwich force-pushed the prestwich/normalize-tracing branch from 71ea1d3 to 1d48594 Compare May 8, 2025 12:46
@prestwich prestwich force-pushed the prestwich/fix-metrics branch from 2f7b513 to cc9f11c Compare May 8, 2025 12:53
@prestwich prestwich force-pushed the prestwich/normalize-tracing branch from 1d48594 to 6050bed Compare May 8, 2025 12:53
Copy link
Member

@Evalir Evalir left a comment

Choose a reason for hiding this comment

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

nice use<>

@prestwich prestwich force-pushed the prestwich/fix-metrics branch from cc9f11c to 710eacb Compare May 8, 2025 18:57
@prestwich prestwich force-pushed the prestwich/normalize-tracing branch from 6050bed to 79a0493 Compare May 8, 2025 18:57
@prestwich prestwich force-pushed the prestwich/fix-metrics branch from 710eacb to f447e78 Compare May 8, 2025 19:05
@prestwich prestwich force-pushed the prestwich/normalize-tracing branch from 79a0493 to 509b9f4 Compare May 8, 2025 19:05
Base automatically changed from prestwich/normalize-tracing to prestwich/new-sdk May 8, 2025 19:05
@prestwich prestwich mentioned this pull request May 8, 2025
Base automatically changed from prestwich/new-sdk to main May 8, 2025 20:53
@prestwich prestwich force-pushed the prestwich/fix-metrics branch from f447e78 to bf796ed Compare May 8, 2025 20:54
@prestwich prestwich enabled auto-merge (squash) May 8, 2025 20:54
@prestwich prestwich merged commit c73021b into main May 8, 2025
5 checks passed
@prestwich prestwich deleted the prestwich/fix-metrics branch May 8, 2025 20:58
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.

3 participants