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

πŸ—ƒοΈ "Personal files" view #42919

Closed
AndyScherzinger opened this issue Jan 18, 2024 · 17 comments Β· Fixed by #43211
Closed

πŸ—ƒοΈ "Personal files" view #42919

AndyScherzinger opened this issue Jan 18, 2024 · 17 comments Β· Fixed by #43211

Comments

@AndyScherzinger
Copy link
Member

AndyScherzinger commented Jan 18, 2024

Context

With heavy use of Group Folders, the All Files menu view is confusing for the user who gets his/her personal files mixing with organization files. Currently there is no option on top level to have filtered view only showing a user their own files.

Describe the solution you'd like

Add an entry Personal files to the left menu, directly below All files, showing:

  • only user files
  • no groupfolders
  • no items shared from other people
@AndyScherzinger AndyScherzinger added enhancement 0. Needs triage Pending check for reproducibility or if it fits our roadmap labels Jan 18, 2024
@AndyScherzinger AndyScherzinger added this to the Nextcloud 29 milestone Jan 18, 2024
@AndyScherzinger AndyScherzinger changed the title Add an entry "My files" to the left menu, showing only user files πŸ—ƒοΈ My files Jan 18, 2024
@jancborchardt jancborchardt changed the title πŸ—ƒοΈ My files πŸ—ƒοΈ "Personal files" view Jan 18, 2024
@jancborchardt
Copy link
Member

jancborchardt commented Jan 18, 2024

Updated title and description a bit, so it's clear it should be "Personal files" instead of "My".

Otherwise good to go, and let's go with an entry then as it's easier to understand than a setting.

@jancborchardt
Copy link
Member

Also cc @tobiasKaminsky on this regarding adding this to mobile apps.

@tobiasKaminsky
Copy link
Member

So it would be list of root with

  • my files (files and folder) that I own
  • all accepted incoming shares

clicking on any of those folders would then show entire content of this folder, or also only limited to above criteria? (would be harder to do)

@sorbaugh
Copy link
Contributor

Assuming, we can already filter files by owner, this should probably be frontend only right? @susnux @Fenn-CS

@skjnldsv
Copy link
Member

@tobiasKaminsky, @sorbaugh edited the first post and lost Jan's description, I added it back :)

@AndyScherzinger
Copy link
Member Author

Thanks for the correction and update of the description in the issue @skjnldsv

That now matches also my understanding of how it is supposed to work / the filter to be done πŸ‘

@susnux
Copy link
Contributor

susnux commented Jan 20, 2024

Assuming, we can already filter files by owner, this should probably be frontend only right?

I would say yes

@emoral435 emoral435 self-assigned this Jan 20, 2024
@sorbaugh sorbaugh removed their assignment Jan 23, 2024
@tobiasKaminsky
Copy link
Member

clicking on any of those folders in "personal view" would then show entire content of this folder, or also only limited to above criteria? (would be harder to do)

Let me re-ask this again :-)

@emoral435
Copy link
Contributor

emoral435 commented Jan 25, 2024

@tobiasKaminsky Just based off of the description, I assuming that it is limited to the above criteria. In specific,

no items shared from other people

So, if my understanding is correct, this would mean that we would have to filter even Files/Nodes within each folder to see if it is shared, etc

@jancborchardt
Copy link
Member

jancborchardt commented Jan 25, 2024

@tobiasKaminsky @emoral435 according to the comment at https://github.com/nextcloud-gmbh/customer-feature-requests/issues/404#issuecomment-1556626596, the root level is the mainly important one.

So since it is likely a lot more work to filter it out at every step, it would make more sense to just start with that? @AndyScherzinger

@AndyScherzinger
Copy link
Member Author

AndyScherzinger commented Jan 25, 2024

So since it is likely a lot more work to filter it out at every step, it would make more sense to just start with that? @AndyScherzinger

Fine by me πŸ‘

Only thing that is not covered then given the referenced comment @jancborchardt https://github.com/nextcloud-gmbh/customer-feature-requests/issues/404#issuecomment-1556626596 is

For a folder shared TO others, this entry should display the content created by others in that folder

and maybe

This entry should display shared file/folder FROM others moved intentionally by the user inside a created/owned folder at root

Simply because I am not sure if this is easily detectable and if the file ownership is actually with the user and not the sharing party. @sorbaugh can you check this with the team.
Also not sure how we view that and if we would consider these "files of the user" because that is what "personal files" as a term reflects for me. Also @jancborchardt for your view on this aspect

@tobiasKaminsky
Copy link
Member

This entry should display shared file/folder FROM others moved intentionally by the user inside a created/owned folder at root

As I understand this, it means

  • folderSharedFromUserAbcToMe is not shown in root folder
  • /myReceivedShares/ is my created folder
  • /myReceivedShares/folderSharedFromUserAbcToMe is then shown within this view

--> then it would also fit in my "assumption" to have this filter only on top level

For a folder shared TO others, this entry should display the content created by others in that folder

Same here. This is shown in root view, and content is also shown, as in subfolder those filter does not apply

@tobiasKaminsky
Copy link
Member

@jancborchardt do we wanna have another icon?

e.g.
image

2024-01-26-081730 2024-01-26-083441

@emoral435
Copy link
Contributor

Just to get a final confirmation from @AndyScherzinger and @jancborchardt , the filter should only apply to the root folder within the personal files view, correct? I agree with Tobias' assumption

@AndyScherzinger
Copy link
Member Author

As far as I read it, yes πŸ˜ŠπŸ‘

@sorbaugh sorbaugh removed the 0. Needs triage Pending check for reproducibility or if it fits our roadmap label Feb 13, 2024
@sorbaugh sorbaugh added 1. to develop Accepted and waiting to be taken care of 3. to review Waiting for reviews and removed 1. to develop Accepted and waiting to be taken care of labels Feb 13, 2024
@sorbaugh
Copy link
Contributor

Adding some details here that might get lost due to the discussion taking place in the PR. There is a breadcrumb optimization that was added to the Personal Files View that needs to be applied to all other Files Views as well for consistency. Will create a separate issue so we can get this issue merged and closed while still tracking the Breadcrumb optimization.

See: #43211 (comment)

@maximelehericy
Copy link

@emoral435 thanks for the work here ! this one has been expected for a long time and will make happy a lot of people :)
I was testing it, and on our cloud I can still see groupfolders under the Personal Files view, which I think should not be there πŸ™ƒ
image

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

Successfully merging a pull request may close this issue.

8 participants