Skip to content

Fix ShareLink Upload UI for Folders - #30256

Merged
artonge merged 2 commits into
masterfrom
fix/allow_edit_folder
Dec 14, 2021
Merged

Fix ShareLink Upload UI for Folders#30256
artonge merged 2 commits into
masterfrom
fix/allow_edit_folder

Conversation

@jotoeri

@jotoerijotoeri commented Dec 14, 2021

Copy link
Copy Markdown
Member

Simple fix of a quite old bug i stumbled upon today. The Upload-options for folders are shown only if the user has permissions to do so. If permissions are not given, until now it fell back to the option for files instead of showing nothing.

Fixes#27962

Corresponding v-if:

<templatev-if="isFolder&&fileHasCreatePermission&&config.isPublicUploadEnabled">

Showing the bug: files option on folder
grafik

Do not show the Upload-Option if it is not allowed to check.
Signed-off-by: Jonas Rittershofer <jotoeri@users.noreply.github.com>
@jotoerijotoeri added 3. to review Waiting for reviews papercut Annoying recurring UX issue with possibly simple fix. labels Dec 14, 2021
@jotoeri

Copy link
Copy Markdown
MemberAuthor

/compile amend/

@jotoeri

Copy link
Copy Markdown
MemberAuthor

/backport to stable23

@jotoeri

Copy link
Copy Markdown
MemberAuthor

/backport to stable22

@jotoeri

Copy link
Copy Markdown
MemberAuthor

/backport to stable21

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

@PVince81PVince81 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.

👍

@artonge
artonge merged commit 069fe1c into masterDec 14, 2021
@artonge
artonge deleted the fix/allow_edit_folder branch December 14, 2021 12:06
@backportbot-nextcloud

Copy link
Copy Markdown

The backport to stable23 failed. Please do this backport manually.

@backportbot-nextcloud

Copy link
Copy Markdown

The backport to stable22 failed. Please do this backport manually.

@backportbot-nextcloud

Copy link
Copy Markdown

The backport to stable21 failed. Please do this backport manually.

@jotoeri

Copy link
Copy Markdown
MemberAuthor

The backport to stable21 failed. Please do this backport manually.

Somebody knows, why this happens? 🤔

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewspapercutAnnoying recurring UX issue with possibly simple fix.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow upload and editing on sharing results in read only

5 participants

@jotoeri@PVince81@artonge@skjnldsv@nextcloud-command