Skip to content

fix: files root detection for public shares - #3005

Merged
skjnldsv merged 2 commits into
masterfrom
fix/undefined
Oct 29, 2025
Merged

fix: files root detection for public shares#3005
skjnldsv merged 2 commits into
masterfrom
fix/undefined

Conversation

@skjnldsv

@skjnldsv skjnldsv commented Oct 28, 2025

Copy link
Copy Markdown
Member

On public shares, we still have a valid root.
this created a discrepancy between the path calculation.

This is more suitable approach to #2991 and #3004

Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
@skjnldsv
skjnldsv requested review from a team, artonge, sorbaugh, susnux and szaimen and removed request for a team October 28, 2025 15:35
@skjnldsv skjnldsv self-assigned this Oct 28, 2025
@skjnldsv skjnldsv added bug Something isn't working regression Regression of a previous working feature high High priority 3. to review Waiting for reviews labels Oct 28, 2025
@skjnldsv skjnldsv added this to the Nextcloud 33 milestone Oct 28, 2025
@skjnldsv

Copy link
Copy Markdown
Member Author

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@skjnldsv

Copy link
Copy Markdown
Member Author

/backport to stable32 please

@skjnldsv

skjnldsv commented Oct 28, 2025

Copy link
Copy Markdown
Member Author

/backport to stable31 please

@backportbot backportbot Bot added the backport-request Pending backport by the backport-bot label Oct 28, 2025
@skjnldsv

Copy link
Copy Markdown
Member Author

/backport to stable30 please

@skjnldsv
skjnldsv merged commit 2f7800e into master Oct 29, 2025
35 of 40 checks passed
@skjnldsv
skjnldsv deleted the fix/undefined branch October 29, 2025 08:12
@backportbot backportbot Bot removed the backport-request Pending backport by the backport-bot label Oct 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug Something isn't working high High priority regression Regression of a previous working feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: files app unable to browse images

4 participants