Skip to content

fix(files_sharing): remove leading slash from filename on unshare toast#61303

Merged
leftybournes merged 2 commits into
masterfrom
leftybournes/fix/sharing-remove-slash-on-unshare-toast
Jun 23, 2026
Merged

fix(files_sharing): remove leading slash from filename on unshare toast#61303
leftybournes merged 2 commits into
masterfrom
leftybournes/fix/sharing-remove-slash-on-unshare-toast

Conversation

@leftybournes

Copy link
Copy Markdown
Member
  • Resolves: #

Summary

Currently, when a file/folder is unshared, the confirmation toast shows the filename with a slash (e.g. File "/Notes.txt" has been unshared). This fixes it so that it no longer contains a slash (e.g. File "Notes.text" has been unshared).

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@leftybournes leftybournes added this to the Nextcloud 35 milestone Jun 15, 2026
@leftybournes leftybournes requested a review from a team as a code owner June 15, 2026 14:23
@leftybournes leftybournes requested review from nfebe, sorbaugh and susnux and removed request for a team June 15, 2026 14:23
@leftybournes leftybournes force-pushed the leftybournes/fix/sharing-remove-slash-on-unshare-toast branch from 69b5711 to 12c0329 Compare June 22, 2026 12:03
@leftybournes leftybournes changed the title fix(files_sharing): remove slash from filename on unshare toast fix(files_sharing): remove leading slash from filename on unshare toast Jun 22, 2026
@susnux susnux requested a review from kra-mo June 22, 2026 12:18

@susnux susnux left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

So basically this removed the root / of the path.

Fine with it if it is confusing for users.

@kra-mo kra-mo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Makes sense, users are not often exposed to this.

Still, doing this as a one-off is never nice, I wonder where else we could do this. Then it could just be displayPath or something.

Signed-off-by: Kent Delante <kent@delante.me>
Signed-off-by: Kent Delante <kent@delante.me>
@leftybournes leftybournes force-pushed the leftybournes/fix/sharing-remove-slash-on-unshare-toast branch from 12c0329 to 5b6139c Compare June 23, 2026 05:27
@leftybournes leftybournes enabled auto-merge June 23, 2026 05:27
@leftybournes

Copy link
Copy Markdown
Member Author

/backport 046de32 to stable34

@leftybournes

Copy link
Copy Markdown
Member Author

/backport 046de32 to stable33

@leftybournes

Copy link
Copy Markdown
Member Author

/backport 046de32 to stable32

@leftybournes leftybournes merged commit c641dfa into master Jun 23, 2026
159 of 162 checks passed
@leftybournes leftybournes deleted the leftybournes/fix/sharing-remove-slash-on-unshare-toast branch June 23, 2026 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants