Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 1 commit into from
Feb 5, 2024
Merged

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

merged 1 commit into from
Feb 5, 2024

Conversation

skjnldsv
Copy link
Member

@skjnldsv skjnldsv commented Feb 3, 2024

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"

@skjnldsv skjnldsv added this to the Nextcloud 29 milestone Feb 3, 2024
@skjnldsv skjnldsv requested review from susnux and a team February 3, 2024 11:41
@skjnldsv skjnldsv self-assigned this Feb 3, 2024
@skjnldsv skjnldsv requested review from Fenn-CS and szaimen and removed request for a team February 3, 2024 11:41
@skjnldsv
Copy link
Member Author

skjnldsv commented Feb 3, 2024

/backport to stable28

Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

🐘

Copy link
Contributor

@emoral435 emoral435 left a comment

Choose a reason for hiding this comment

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

🐘

@skjnldsv
Copy link
Member Author

skjnldsv commented Feb 5, 2024

/compile rebase-amend /

@skjnldsv skjnldsv 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
Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@skjnldsv skjnldsv merged commit d7dc0f3 into master Feb 5, 2024
97 checks passed
@skjnldsv skjnldsv deleted the fix/ownerid-string branch February 5, 2024 15:18
@ChildLearningClub
Copy link

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
Copy link
Contributor

susnux commented Apr 3, 2024

@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
Copy link

@susnux Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish bug feature: files technical debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: No more files after update to 28.0.2
5 participants