Skip to content

[stable33] fix(files): correctly handle nodes with snowflake ids#58999

Merged
susnux merged 4 commits intostable33from
backport/58611/stable33
Mar 17, 2026
Merged

[stable33] fix(files): correctly handle nodes with snowflake ids#58999
susnux merged 4 commits intostable33from
backport/58611/stable33

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Mar 17, 2026

Backport of #58611

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot backportbot bot added bug 3. to review Waiting for reviews labels Mar 17, 2026
@backportbot backportbot bot added this to the Nextcloud 33.0.1 milestone Mar 17, 2026
susnux added 4 commits March 17, 2026 12:55
We introduced `id` instead of `fileid` to be always of type `string` to
allow snow flake ids. So we need to check the new attribute instead of
the legacy one.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux force-pushed the backport/58611/stable33 branch from 7fea5e0 to ec3de81 Compare March 17, 2026 11:57
@susnux susnux marked this pull request as ready for review March 17, 2026 11:57
@susnux susnux requested a review from a team as a code owner March 17, 2026 11:57
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.

🐘

@susnux susnux merged commit 40db2a4 into stable33 Mar 17, 2026
127 of 130 checks passed
@susnux susnux deleted the backport/58611/stable33 branch March 17, 2026 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants