Skip to content

fix(files): add mount root property and adjust delete wording - #43217

Merged
skjnldsv merged 2 commits into
masterfrom
fix/sharing-delee-leave-share
Feb 6, 2024
Merged

fix(files): add mount root property and adjust delete wording#43217
skjnldsv merged 2 commits into
masterfrom
fix/sharing-delee-leave-share

Conversation

@skjnldsv

@skjnldsvskjnldsv commented Jan 30, 2024

Copy link
Copy Markdown
Member

Follow-up of #43152
Fix#40940

image
image

@skjnldsvskjnldsv added this to the Nextcloud 29 milestone Jan 30, 2024
@skjnldsv
skjnldsv requested review from a team, artonge and icewind1991January 30, 2024 16:29
@skjnldsvskjnldsv self-assigned this Jan 30, 2024
@skjnldsv
skjnldsv requested review from nfebe and sorbaugh and removed request for a teamJanuary 30, 2024 16:29
Comment on lines +34 to +36
registerDavProperty('nc:share-attributes', { nc: 'http://nextcloud.org/ns' })
registerDavProperty('oc:share-types', { oc: 'http://owncloud.org/ns' })
registerDavProperty('ocs:share-permissions', { ocs: 'http://open-collaboration-services.org/ns' })

@skjnldsvskjnldsvJan 30, 2024

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

We should remove them from the default fetch, files_sharing might be disabled

@skjnldsv
skjnldsvforce-pushed the fix/sharing-delee-leave-share branch 2 times, most recently from 3aa1b86 to c91ca33CompareJanuary 30, 2024 16:41
Comment threadapps/files/src/init.ts
@skjnldsv
skjnldsvforce-pushed the fix/sharing-delee-leave-share branch 2 times, most recently from 01c9425 to 88ec1dcCompareJanuary 31, 2024 10:59
@skjnldsv
skjnldsv requested a review from artongeJanuary 31, 2024 11:00
Comment threadapps/files/src/actions/deleteAction.ts Outdated
Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
@skjnldsv
skjnldsvforce-pushed the fix/sharing-delee-leave-share branch from 6309b59 to ca61b11CompareFebruary 6, 2024 16:44
@skjnldsvskjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Feb 6, 2024
@skjnldsv

Copy link
Copy Markdown
MemberAuthor

/backport 6cf4fe1 to stable28

@skjnldsv
skjnldsv enabled auto-merge February 6, 2024 16:46
@skjnldsv
skjnldsv disabled auto-merge February 6, 2024 18:44
@skjnldsv
skjnldsv merged commit 7b4914b into masterFeb 6, 2024
@skjnldsv
skjnldsv deleted the fix/sharing-delee-leave-share branch February 6, 2024 19:25
@blizzzblizzz mentioned this pull request Mar 5, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishbugfeature: external storagefeature: filesfeature: sharingtechnical debt🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[F2V] Disconnect storage and Leave this share

3 participants

@skjnldsv@artonge@come-nc