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

Indirect share info and owner missing in flat lists #3040

Closed
1 task
PVince81 opened this issue Feb 14, 2020 · 1 comment · Fixed by #3135
Closed
1 task

Indirect share info and owner missing in flat lists #3040

PVince81 opened this issue Feb 14, 2020 · 1 comment · Fixed by #3135
Assignees
Labels
Type:Bug Something isn't working

Comments

@PVince81
Copy link
Contributor

PVince81 commented Feb 14, 2020

Steps

  1. Login as "admin"
  2. Create a folder "test"
  3. Share "test" with another user "user1"
  4. Create a subfolder "test/sub"
  5. Open the share panel for "test/sub" and remember the entries
  6. Mark "test/sub" as favorite
  7. Go to the "Favorites" list
  8. Click the "sub" entry to open the sharing panel
  9. Refresh the page
  10. Click "sub" entry again

Expected

In the favorites page, the same entries must be visible like for the "All files" page.
This means the share owner, the collaborators and the indirect collaborators.
Same for links.

Actual

On first access, you might see the correct entries because of caching.
After refresh, only the direct collaborators are shown.

Version

0.4.0

Technical

  • load sharesTree and incomingShares in state for all parents of the given entry when opening sidebar (and not already cached/populated)
@PVince81 PVince81 added the Type:Bug Something isn't working label Feb 14, 2020
@PVince81 PVince81 self-assigned this Mar 5, 2020
@PVince81
Copy link
Contributor Author

PVince81 commented Mar 5, 2020

PR here: #3135

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type:Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant