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

Makefile: Update Cockpit lib to 140b7e62f5cfcad5ad9105867474526ef68351db #1312

Merged
merged 2 commits into from
Nov 16, 2023

Conversation

github-actions[bot]
Copy link
Contributor

No description provided.

@github-actions github-actions bot added the bot label Nov 16, 2023
@github-actions github-actions bot changed the title [no-test] Makefile: Update Cockpit lib to 9da922928c1b1ea8489e12ba5ba6ecfd Makefile: Update Cockpit lib to 9da922928c1b1ea8489e12ba5ba6ecfd Nov 16, 2023
@github-actions github-actions bot force-pushed the cockpit-lib-update-cockpit-lib-20231116-022114 branch from 843a423 to 84c4abd Compare November 16, 2023 02:21
@martinpitt martinpitt self-assigned this Nov 16, 2023
@martinpitt
Copy link
Member

A-ha! Dialogs on top of each other! That's actually what I wanted to check for last week already after cockpit-project/cockpit#19595 , but didn't get around to it. Looking..

@martinpitt martinpitt force-pushed the cockpit-lib-update-cockpit-lib-20231116-022114 branch from 84c4abd to 13c0859 Compare November 16, 2023 10:31
@martinpitt martinpitt changed the title Makefile: Update Cockpit lib to 9da922928c1b1ea8489e12ba5ba6ecfd Makefile: Update Cockpit lib to 140b7e62f5cfcad5ad9105867474526ef68351db Nov 16, 2023
@martinpitt martinpitt force-pushed the cockpit-lib-update-cockpit-lib-20231116-022114 branch from 13c0859 to 2c6f469 Compare November 16, 2023 12:02
@martinpitt martinpitt marked this pull request as draft November 16, 2023 12:02
martinpitt and others added 2 commits November 16, 2023 13:13
Close the dialog in the "does not oops" and "add disk xfail" checks,
otherwise the test continues fumbling the main page and opening new
dialogs underneath the existing open one. That's cheating and a source
for race conditions.

Fix other places which close dialogs but don't wait for it to go away.

This uncovered a place which wants to open a new "add disk" dialog, and
forgot to switch to "Custom path" mode. This worked before as the
previous dialog was still open and happened to be in that mode.
@martinpitt martinpitt force-pushed the cockpit-lib-update-cockpit-lib-20231116-022114 branch from 2c6f469 to 1e0943b Compare November 16, 2023 12:13
@martinpitt martinpitt marked this pull request as ready for review November 16, 2023 12:35
@jelly jelly merged commit f30caa3 into main Nov 16, 2023
27 checks passed
@jelly jelly deleted the cockpit-lib-update-cockpit-lib-20231116-022114 branch November 16, 2023 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants