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

Fixes #37758 - Delete host redirects to hosts list based on setting #10424

Merged
merged 1 commit into from
Jan 30, 2025

Conversation

lfu
Copy link
Contributor

@lfu lfu commented Jan 28, 2025

Test steps

  1. All Hosts - Action - Delete on one of the hosts
  2. All Hosts - click on one host - click on kebab menu on the right - Delete
    After deleting a host, the page should be redirected to the All Hosts page based on the setting of Show new host overview page.

@lfu lfu force-pushed the 29347_new_hosts_page branch from df33ad4 to dbef310 Compare January 28, 2025 19:09
@lfu
Copy link
Contributor Author

lfu commented Jan 29, 2025

Don't think the test failure is related.

Copy link
Member

@MariaAga MariaAga left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@MariaAga MariaAga merged commit 07ff0b6 into theforeman:develop Jan 30, 2025
36 of 39 checks passed
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.

2 participants