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

[web] Shared with me: Show / Hide shares #9531

Closed
exalate-issue-sync bot opened this issue Aug 3, 2023 · 5 comments · Fixed by #9718
Closed

[web] Shared with me: Show / Hide shares #9531

exalate-issue-sync bot opened this issue Aug 3, 2023 · 5 comments · Fixed by #9718
Assignees

Comments

@exalate-issue-sync
Copy link

exalate-issue-sync bot commented Aug 3, 2023

Description

User Stories

  • As a user I want to hide unwanted shares so that I can keep an overview over my shares.

  • As a user I want to unsync shares so that my local disk space does not get full.

  • As a user I want to sync selected shares so that I can access these files on my devices.

Value

Acceptance Criteria

  • All incoming shares are initally shown in "Shared with me" (no more "Accepted- Pending- and Declined shares sections")
  • All shares have a new status "visible" or "hidden" (so that users can tidy up unwanted shares); default is "visible"
  • oc10: status visible/hidden is not existent and all accepted/declined shares are shown in a single view.
  • click on "hide" on a share hides the share
  • click on "unhide" shows the share
  • Button group "Shares" and "Hidden Shares"
    • Click on the respective button shows either shares with the status "hidden" or "unhidden"
  • If autoaccept is serverside
    • enabled, all shares are synced with the mobile& desktop clients
    • disabled, all shares are not automatically available in the mobile and desktop clients
  • New button per share: "Enable sync" / "Disable sync" syncs/unsync the share with mobile & desktop clients
    • If a share is "hidden", it still can get get enabled/disabled for syncing (cc @elizavetaRa @diocas @pascalwengerter )
    • "Enable sync" = ~ "Accept share"
    • "Disable sync" = ~ "Pending" for initial incoming shares
    • "Disable sync" = ~ "Decline" if share is not in initial pending state

Definition of ready

[ ] everybody needs to understand the value written in the user story
[ ] acceptance criteria has to be defined
[ ] all dependencies of the user story need to be identified
[ ] feature should be seen from an end user perspective
[ ] user story has to be estimated
[ ] story points need to be less then 20

Definition of done

  • Functional requirements
    [ ] functionality described in the user story works
    [ ] acceptance criteria are fulfilled
  • Quality
    [ ] code review happened
    [ ] CI is green
    [ ] critical code received unit tests by the developer
    [ ] automated tests passed (if automated tests are not available, this test needs to be created and passed
  • Non-functional requirements
    [ ] no sonar cloud issues

Screenshots

Initial View

  • Shows visible shares
  • quickaction to Hide shares
  • Status column shows, if Sync with devices is enabled
  • enable sync via batch action or context menu

image

Hidden Shares

  • Toggle button shows hidden shares
  • If a share gets hidden, it can still get snyced/unsynced.
    image
@exalate-issue-sync
Copy link
Author

Florian Schade commented: [~mstingl] FYI, maybe for the clients?

@tbsbdr
Copy link
Contributor

tbsbdr commented Sep 28, 2023

Updated ACs in initial post

  • If a share is "hidden", it still can get get enabled/disabled for syncing
  • Updated wording (in consultation of @TheOneRing )
    • Sync Unsync -> Enable sync Disable sync

@pascalwengerter
Copy link
Contributor

@tbsbdr which icons from https://owncloud.design/#/Design%20Tokens/IconList to use for Enable sync/Disable sync actions? Or do you have custom ones in mind?

@tbsbdr
Copy link
Contributor

tbsbdr commented Sep 29, 2023

hm.. yeah. a custom one would be good but for now I'd propose to use computer-line (for both, enable and disable)
screenshot_001728

@tbsbdr
Copy link
Contributor

tbsbdr commented Oct 6, 2023

Note from weekly exchange cern & oc:

  • we discussed to go with the approach "If a share is 'hidden', it still can get get enabled/disabled for syncing"
  • it could be useful to provide a dialog: click: hide share -> dialog "Do you want to disable sync? [yes] [No]" but this dialog is not part of this story - first we would like to observe how the "hide" and "sync" options are getting adapted by users

@micbar micbar modified the milestone: CERN Web Merge Oct 9, 2023
@tbsbdr tbsbdr moved this from Backlog to In progress in Infinite Scale Team Board Oct 24, 2023
JammingBen added a commit that referenced this issue Oct 30, 2023
* Next iteration of share show/hide

* Rebase and add changelog item

* put remaining parts together to make it work

* fixup! put remaining parts together to make it work

* keep original action names internally for accepting/declining shares

* test: fix tests

* do not allow decline action on pending shares

* feat: rename hide to hidden, use PUT for the request to set it

* fix: center no-content-message for shared-with-me-section

* feat: add action to hide shares to context menu

* fix: ensure the share hidden-param does not get sent when not needed

* fix pipeline

---------

Co-authored-by: Jannik Stehle <[email protected]>
@github-project-automation github-project-automation bot moved this from In progress to Done in Infinite Scale Team Board Oct 30, 2023
AlexAndBear pushed a commit that referenced this issue Dec 13, 2023
* Next iteration of share show/hide

* Rebase and add changelog item

* put remaining parts together to make it work

* fixup! put remaining parts together to make it work

* keep original action names internally for accepting/declining shares

* test: fix tests

* do not allow decline action on pending shares

* feat: rename hide to hidden, use PUT for the request to set it

* fix: center no-content-message for shared-with-me-section

* feat: add action to hide shares to context menu

* fix: ensure the share hidden-param does not get sent when not needed

* fix pipeline

---------

Co-authored-by: Jannik Stehle <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants