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

"Are you sure?" before delete business process itself #431

Open
xeiss opened this issue Nov 28, 2023 · 3 comments
Open

"Are you sure?" before delete business process itself #431

xeiss opened this issue Nov 28, 2023 · 3 comments
Labels
enhancement New feature or improvement

Comments

@xeiss
Copy link

xeiss commented Nov 28, 2023

Describe the bug

You can easy miss click in the config of a business process and delete it with one click.

To Reproduce

  1. Create a new Business Process
  2. Store it
  3. Click on Config
  4. Miss click on Delete
  5. Image this was a business process with many nodes inside and you have no backup, because you created it some minutes before

Expected behavior

"Are you sure?" Question before deleting a whole business process, as it is already implemented when I want to delete a Host/Service/Process Node

Your Environment

  • Icinga Web 2 version and modules (System - About): 2.12.1
  • Web browser used: Firefox
  • Icinga 2 version used (icinga2 --version): r2.14.0-1
  • PHP version used (php --version): 8.0
  • Server operating system and version: Debian 11
@nilmerg nilmerg added the enhancement New feature or improvement label Jan 8, 2024
@nilmerg
Copy link
Member

nilmerg commented Jan 8, 2024

You have to store your change first. Should be enough time to notice the mishap. But in a sense I can comprehend why you want this. Though personally, I'd want this only for processes, not nodes.

@xeiss
Copy link
Author

xeiss commented Jan 8, 2024

I think you misunderstood. I don't mean a processes or a nodes delete, the are fine.
But you can delete the complete "business process" on top level with a single click in config, there is no store your changes after this.

@nilmerg
Copy link
Member

nilmerg commented Jan 9, 2024

Ooooh, yeah, that makes even more sense!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or improvement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants