8000 Canceled acceleration on timeline by natsoni · Pull Request #5671 · mempool/mempool · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Canceled acceleration on timeline #5671

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 2 commits into from
Dec 8, 2024
Merged

Conversation

natsoni
Copy link
Contributor
@natsoni natsoni commented Dec 5, 2024

Builds on #5668

Acceleration timeline on canceled accelerations:

Screen.Recording.2024-12-04.at.18.52.31.mov

If / when the canceled acceleration is confirmed:

Screenshot 2024-12-05 at 13 05 04

@natsoni natsoni requested a review from wiz December 5, 2024 12:06
@cla-bot cla-bot bot added the cla-signed label Dec 5, 2024
@natsoni natsoni changed the title Natsoni/cancelled accel on timeline Cancelled acceleration on timeline Dec 5, 2024
@natsoni natsoni changed the title Cancelled acceleration on timeline Canceled acceleration on timeline Dec 5, 2024
@wiz wiz requested a review from mononaut December 6, 2024 08:11
Copy link
Member
@wiz wiz left a comment

Choose a reason for hiding this comment

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

utACK

@wiz
Copy link
Member
wiz commented Dec 8, 2024
     AssertionError: Timed out retrying after 4000ms: Expected <div.spinner-border.text-light> not to exist in the DOM, but it was continuously found.

looks like a failing test, but will merge anyway

@wiz wiz merged commit 0d31143 into master Dec 8, 2024
33 of 37 checks passed
@wiz wiz deleted the natsoni/cancelled-accel-on-timeline branch December 8, 2024 04:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0