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

[dev.icinga.com #8584] Provide a "Select all" link in the monitoring list views #1415

Closed
icinga-migration opened this issue Mar 3, 2015 · 19 comments
Labels
area/monitoring Affects the monitoring module duplicate We already handle this in a different issue enhancement New feature or improvement

Comments

@icinga-migration
Copy link

This issue has been migrated from Redmine: https://dev.icinga.com/issues/8584

Created by mjentsch on 2015-03-03 14:38:37 +00:00

Assignee: (none)
Status: New
Target Version: Backlog
Last Update: 2016-12-20 15:20:38 +00:00 (in Redmine)


When in list view, it should be possible to select all hosts/services in the list to perform actions on them. This was already implemented some time ago but got removed again.

Note: This feature was also requested by some people during the last Webinar, and I agree that this would be a nice feature.


Relations:

@icinga-migration
Copy link
Author

Updated by mjentsch on 2015-03-03 14:38:53 +00:00

  • Category set to UI

@icinga-migration
Copy link
Author

Updated by elippmann on 2015-03-03 22:52:11 +00:00

  • Subject changed from List: Add "select all" to list views. to Provide a "Select all" link in the monitoring list views
  • Category changed from UI to Monitoring
  • Target Version set to 2.0.0-beta3

@icinga-migration
Copy link
Author

Updated by tgelf on 2015-03-03 23:33:55 +00:00

I have strong objections against this one, here a few of them:

  • we still fail when it goes to select a few hundred rows as of URL length restrictions
  • multiline selection does not work across pages
  • given those, what das "all" mean to you?

I could live with a link allowing me to continue with all objects matched by the current filter. And I see no quick way to achieve a consistent behaviour otherwise.

Cheers,
Thomas

@icinga-migration
Copy link
Author

Updated by mjentsch on 2015-03-05 14:46:35 +00:00

tgelf wrote:

I have strong objections against this one, here a few of them:

* we still fail when it goes to select a few hundred rows as of URL length restrictions
* multiline selection does not work across pages
* given those, what das "all" mean to you?

All services that are matched by the current list filter.

I could live with a link allowing me to continue with all objects matched by the current filter. And I see no quick way to achieve a consistent behaviour otherwise.

Yes, if we implement this the multi-views and the command-forms need to understand regular filters, otherwise the URLs will get too long (and ugly). The old implementation just selected all elements in the current view, I don't think that this was in any way useful, which is why it was removed.

@icinga-migration
Copy link
Author

Updated by elippmann on 2015-03-05 15:35:51 +00:00

There's absolutely nothing which speaks against this feature. It just means that we open hosts/show and services/show w/ the current filter. Our command forms then have to be optimised to not show all involved hosts or services. Multi-selection across pages and "continue w/ this filter" URLs are separate issues.

@icinga-migration
Copy link
Author

Updated by tgelf on 2015-03-05 15:38:46 +00:00

It's just about wording. "Select all" implies something we cannot ship for mentioned reasons. An action named "Use/continue with filtered objects" is what we can easily provide.

Cheers,
Thomas

@icinga-migration
Copy link
Author

Updated by icinga-kanban on 2015-03-05 17:37:32 +00:00

Build !#544 triggered by commit 860cc59 failed.

Branch: origin/feature/improve-multi-select-view-8584
Author: Matthias Jentsch

@icinga-migration
Copy link
Author

Updated by icinga-kanban on 2015-03-06 12:36:12 +00:00

Build !#546 triggered by the commits f91ce0c, a077472, 0623404, 9fc80ec, 7153436, f425704, cb0ca6d, ef16ba5 failed.

Branch: origin/feature/improve-multi-select-view-8584
Author: Matthias Jentsch

@icinga-migration
Copy link
Author

Updated by icinga-kanban on 2015-03-06 13:54:52 +00:00

Build !#547 triggered by commit 4724eb9 failed.

Branch: origin/feature/improve-multi-select-view-8584
Author: Matthias Jentsch

@icinga-migration
Copy link
Author

Updated by icinga-kanban on 2015-03-06 15:16:09 +00:00

Build !#548 triggered by the commits 9c45e99, 2591f05, 5c71f5a, c2c9e49 failed.

Branch: origin/feature/improve-multi-select-view-8584
Author: Matthias Jentsch

@icinga-migration
Copy link
Author

Updated by icinga-kanban on 2015-03-06 16:36:10 +00:00

Build !#549 triggered by commit 1cfcb93 failed.

Branch: origin/feature/improve-multi-select-view-8584
Author: Matthias Jentsch

@icinga-migration
Copy link
Author

Updated by icinga-kanban on 2015-03-06 16:55:22 +00:00

Build !#550 triggered by commit 266e781 failed.

Branch: origin/feature/improve-multi-select-view-8584
Author: Matthias Jentsch

@icinga-migration
Copy link
Author

Updated by icinga-kanban on 2015-03-06 17:07:34 +00:00

Build !#551 triggered by commit 95a83a4 failed.

Branch: origin/feature/improve-multi-select-view-8584
Author: Matthias Jentsch

@icinga-migration
Copy link
Author

Updated by elippmann on 2015-03-13 02:43:23 +00:00

  • Target Version changed from 2.0.0-beta3 to 2.0.0-rc1

@icinga-migration
Copy link
Author

Updated by icinga-kanban on 2015-03-31 15:47:05 +00:00

Build !#575 triggered by commit 9fbd782 failed.

Branch: origin/feature/improve-multi-select-view-8584
Author: Matthias Jentsch

@icinga-migration
Copy link
Author

Updated by jmeyer on 2015-06-11 07:45:57 +00:00

  • Target Version changed from 2.0.0-rc1 to Backlog

@icinga-migration
Copy link
Author

Updated by aklimov on 2016-12-20 15:20:38 +00:00

This will be fixed once a88822c has been merged.

@icinga-migration
Copy link
Author

Updated by aklimov on 2016-12-20 15:21:00 +00:00

  • Blocked set to 11309

@icinga-migration icinga-migration added enhancement New feature or improvement area/monitoring Affects the monitoring module labels Jan 17, 2017
@icinga-migration icinga-migration added this to the Backlog milestone Jan 17, 2017
@lippserd
Copy link
Member

I close this as duplicate of #2344 because there is a pending PR.

@lippserd lippserd added the duplicate We already handle this in a different issue label Oct 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/monitoring Affects the monitoring module duplicate We already handle this in a different issue enhancement New feature or improvement
Projects
None yet
Development

No branches or pull requests

2 participants