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

Shares View: Grouping of Shares #8423

Closed
dragotin opened this issue Feb 13, 2023 · 2 comments
Closed

Shares View: Grouping of Shares #8423

dragotin opened this issue Feb 13, 2023 · 2 comments

Comments

@dragotin
Copy link
Contributor

dragotin commented Feb 13, 2023

Summary

  • Now: Add grouping (default: off, activate via config)
  • Later: Replace grouping-feature by filters

Problem description

For installations with may many shares, the current "Shares" page might be not clean and structured enough.

Proposed solution

@elizavetaRa from CERN has made a proposal with a grouped table which allows to structure the output of the shares tables based on attributes of the shares, ex. the owner, via a combo box.

This could make it's way into ownCloud Web before it gets finally replaced or combined with a solution based on filtering as proposed by @tbsbdr

  • Grouping-feature should be configurable
  • Grouping-feature should be disabled by default

image

Alternative solutions you considered

  • Multi Select Combobox with search option

Reasoning:

Mainly efforts:

  • Rewrite Simple filtering is less expensive to implement (current code works and suits perfectly for cernbox users, but we'd need a rewrite of the code if we'd like to productize it) @kulmann please correct me if I'm wrong
  • Lower maintenance costs on the long run as filters are a re-used component in Web we already need to maintain

image

@tbsbdr tbsbdr changed the title Structured Shares Page Shares View: Group Shares Feb 14, 2023
@tbsbdr
Copy link
Contributor

tbsbdr commented Feb 14, 2023

(Updated initial comment of @dragotin to have all info in one place)

@tbsbdr tbsbdr changed the title Shares View: Group Shares Shares View: Grouping of Shares Jun 1, 2023
@JammingBen
Copy link
Contributor

Has been implemented via #9438.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants