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.

Sign up for GitHub

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

Jump to bottom

fix: fix access to version trough shares #3089

Merged
merged 1 commit into from
Aug 2, 2024
Merged

fix: fix access to version trough shares #3089

merged 1 commit into from
Aug 2, 2024

Conversation

icewind1991
Copy link
Member

To test"

  • Create groupfolder gf1 and give userA access
  • As userA create gf1/test.txt and edit it to create a version history
  • As userA share gf1/test.txt to userB
  • As userB try to access the version history of test.txt

@icewind1991 icewind1991 added the 3. to review Items that need to be reviewed label Jul 30, 2024
@icewind1991 icewind1991 requested review from artonge and come-nc July 30, 2024 16:24
@icewind1991 icewind1991 mentioned this pull request Jul 30, 2024
Closed
@icewind1991
fix: fix access to version trough shares
d72cdef 
Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991 icewind1991 merged commit 7e377d7 into master Aug 2, 2024
42 checks passed
@icewind1991 icewind1991 deleted the shared-version branch August 2, 2024 20:33
@icewind1991
Copy link
Member Author

/backport to stable29

@backportbot backportbot bot mentioned this pull request Aug 7, 2024
Merged
@icewind1991
Copy link
Member Author

/backport to stable28

@backportbot backportbot bot mentioned this pull request Aug 7, 2024
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@come-nc come-nc come-nc approved these changes

@artonge artonge Awaiting requested review from artonge

Assignees
No one assigned
Labels
3. to review Items that need to be reviewed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@icewind1991 @come-nc