Skip to content

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

Open
leftybournes wants to merge 1 commit into
masterfrom
leftybournes/fix/sharing-remove-slash-on-unshare-toast
Open

fix(files_sharing): remove leading slash from filename on unshare toast#61303
leftybournes wants to merge 1 commit 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
Signed-off-by: Kent Delante<kent@delante.me>
@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.

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