Skip to content

fix(files): dav owner and mime types - #43318

Merged
skjnldsv merged 1 commit into
masterfrom
fix/ownerid-string
Feb 5, 2024
Merged

fix(files): dav owner and mime types#43318
skjnldsv merged 1 commit into
masterfrom
fix/ownerid-string

Conversation

@skjnldsv

@skjnldsvskjnldsv commented Feb 3, 2024

Copy link
Copy Markdown
Member

Library fix in nextcloud-libraries/nextcloud-files#882
Partially fix#43269 (comment)

[ERROR] files: Error while fetching content Object { app: "files", uid: "21341", level: 0, error: Error }
app: "files"
error: Error: Invalid owner type
uid: "21341"

@skjnldsvskjnldsv added this to the Nextcloud 29 milestone Feb 3, 2024
@skjnldsv
skjnldsv requested review from a team and susnuxFebruary 3, 2024 11:41
@skjnldsvskjnldsv self-assigned this Feb 3, 2024
@skjnldsv
skjnldsv requested review from nfebe and szaimen and removed request for a teamFebruary 3, 2024 11:41
@skjnldsv

skjnldsv commented Feb 3, 2024

Copy link
Copy Markdown
MemberAuthor

/backport to stable28

@skjnldsvskjnldsv mentioned this pull request Feb 3, 2024
Closed
8 tasks
@skjnldsv
skjnldsv requested a review from artongeFebruary 3, 2024 11:45
szaimen
szaimen approved these changes Feb 5, 2024
View reviewed changes

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

🐘

emoral435
emoral435 approved these changes Feb 5, 2024
View reviewed changes

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

🐘

@skjnldsv

skjnldsv commented Feb 5, 2024

Copy link
Copy Markdown
MemberAuthor

/compile rebase-amend /

@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 5, 2024
@skjnldsv
skjnldsv enabled auto-merge February 5, 2024 14:44
@skjnldsv@nextcloud-command
fix(files): dav owner and mime types
77f307b
Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@skjnldsv
skjnldsv disabled auto-merge February 5, 2024 14:56
@skjnldsv
skjnldsv merged commit d7dc0f3 into masterFeb 5, 2024
@skjnldsv
skjnldsv deleted the fix/ownerid-string branch February 5, 2024 15:18
@backportbotbackportbotBot mentioned this pull request Feb 5, 2024
Merged
2 tasks
@joshtrichardsjoshtrichards mentioned this pull request Feb 8, 2024
Closed
8 tasks
@joshtrichardsjoshtrichards mentioned this pull request Feb 9, 2024
Closed
8 tasks
@blizzzblizzz mentioned this pull request Mar 5, 2024
Merged
@cben0istcben0ist mentioned this pull request Apr 2, 2024
Closed
8 tasks
@ChildLearningClub

ChildLearningClub commented Apr 2, 2024

Copy link
Copy Markdown

Hi, I also have an account with just numbers and upgrading to 28.04 fixed Folders displaying within Nextcloud with your patch, thank you.

I am however also experiencing a similar issue when attempting to add additional paths to Timeline view within the Memories app. The console window provides a very similar error. Is this the same issue by chance?

Additionally, I do not have this same problem within my other account with a username that is not just numbers.

Screenshot 2024-04-03 032941

@susnux

susnux commented Apr 3, 2024

Copy link
Copy Markdown
Contributor

@ChildLearningClub best you report this on the Memories app. Here: https://github.com/pulsejet/memories/issues

Might be that they use an old library version.

@ChildLearningClub

ChildLearningClub commented Apr 5, 2024

Copy link
Copy Markdown

@susnux Thank you.

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

Reviewers

@susnuxsusnuxsusnux approved these changes
@szaimenszaimenszaimen approved these changes
@nfebenfebeAwaiting requested review from nfebenfebe is a code owner automatically assigned from nextcloud/server-frontend
@artongeartongeAwaiting requested review from artonge
+1 more reviewer
@emoral435emoral435emoral435 approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@skjnldsvskjnldsv

Labels

4. to releaseReady to be released and/or waiting for tests to finishbugfeature: filestechnical debt🧱 🤔🚀

Projects

None yet

Milestone

Nextcloud 29

Development

Successfully merging this pull request may close these issues.

[Bug]: No more files after update to 28.0.2

5 participants

@skjnldsv@ChildLearningClub@susnux@szaimen@emoral435