Skip to content

fix(snowflake): cast lastId to string#58206

Merged
miaulalala merged 1 commit intomasterfrom
fix/preview-637/cast-snowflake-id-to-int
Feb 10, 2026
Merged

fix(snowflake): cast lastId to string#58206
miaulalala merged 1 commit intomasterfrom
fix/preview-637/cast-snowflake-id-to-int

Conversation

@miaulalala
Copy link
Contributor

@miaulalala miaulalala commented Feb 9, 2026

Fixes nextcloud/previewgenerator#637

Summary

Snowflake IDs are strings, so the mapper method needs to change

TODO

  • ...

Checklist

@miaulalala miaulalala added this to the Nextcloud 34 milestone Feb 9, 2026
@miaulalala miaulalala self-assigned this Feb 9, 2026
@miaulalala miaulalala requested a review from a team as a code owner February 9, 2026 17:31
@miaulalala miaulalala requested review from ArtificialOwl and icewind1991 and removed request for a team February 9, 2026 17:31
@miaulalala miaulalala added the bug label Feb 9, 2026
@miaulalala miaulalala added 3. to review Waiting for reviews 33-feedback labels Feb 9, 2026
@miaulalala
Copy link
Contributor Author

/backport to stable33

@AndyXheli
Copy link
Contributor

Tested on server and i can confirm this fixes the issue

@miaulalala miaulalala force-pushed the fix/preview-637/cast-snowflake-id-to-int branch from 6ab3ada to c536fe4 Compare February 9, 2026 17:56
Signed-off-by: Anna Larch <anna@nextcloud.com>
@miaulalala miaulalala force-pushed the fix/preview-637/cast-snowflake-id-to-int branch from c536fe4 to 99a1150 Compare February 9, 2026 20:36
@miaulalala miaulalala requested a review from CarlSchwan February 9, 2026 20:37
@miaulalala miaulalala changed the title fix(snowflake): cast lastId to int fix(snowflake): cast lastId to string Feb 10, 2026
@miaulalala miaulalala enabled auto-merge February 10, 2026 10:12
@miaulalala miaulalala merged commit 8c01737 into master Feb 10, 2026
174 checks passed
@miaulalala miaulalala deleted the fix/preview-637/cast-snowflake-id-to-int branch February 10, 2026 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

occ preview:cleanup Peview:cleanup An unhandled exception has been thrown:

4 participants