Skip to content

Conversation

@ScharfViktor
Copy link
Contributor

related #1523

Copilot AI review requested due to automatic review settings November 5, 2025 10:52
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds test coverage for downloading folders in public links for both authenticated and anonymous users, addressing issue #1712. The changes extend an existing test scenario to verify folder download functionality through public links.

  • Adds a new test user "Edith" to the scenario
  • Implements test cases for downloading folders via public links for authenticated users
  • Implements test cases for downloading folders via public links for anonymous users

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@JammingBen
Copy link
Contributor

@ScharfViktor The fix has been merged, you can rebase this PR now.

@ScharfViktor ScharfViktor force-pushed the downloadFolderInpublicLink branch from 9bedc3a to 48ddc5e Compare November 11, 2025 07:24
Copy link
Contributor

@JammingBen JammingBen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@JammingBen JammingBen merged commit 89326f9 into main Nov 11, 2025
28 checks passed
@JammingBen JammingBen deleted the downloadFolderInpublicLink branch November 11, 2025 07:57
openclouders pushed a commit that referenced this pull request Nov 11, 2025
@openclouders openclouders mentioned this pull request Nov 11, 2025
1 task
@openclouders openclouders mentioned this pull request Dec 15, 2025
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants